feat: Causal dmlestimator #8
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n LinearDMLEstimator
* Resolve comments for LinearDMLEstimator * use pipeline in trainInternal
…e it between DML estimator and model; use vector_to_array instead of our own udf
…s issue by creating new instance of model
|
Hey @dylanw-oss 👋! We use semantic commit messages to streamline the release process. Examples of commit messages with semantic prefixes:
To test your commit locally, please follow our guild on building from source. |
dylanw-oss
added a commit
that referenced
this pull request
Jan 3, 2023
* feat: Causal dmlestimator (#8) Add package 'com.microsoft.azure.synapse.ml.causal' and implementation LinearDMLEstimator * resolve comments and fix build pipeline failures * add //scalastyle:off method.length for trainIternal * fix DML Python API missing problem * fix build pipeline failures * rename LinearDMLEstimator to DoubleMLEstimator * rename DML APIs * forgot to change DoubleMLModel name in fuzzingTest * use findUnusedColumnName for treatmentResidual and outcomeResidul comlumns * address a few small comments * remove HasExcludeFeatureCols and use HasInputCols * fix style issue, Params.scala: File must end with newline character * set DoubleMLEstimator transform as experimental Co-authored-by: Jason Wang <jasowang@microsoft.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issues/PRs
#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?
Does this PR change any dependencies?
Does this PR add a new feature? If so, have you added samples on website?
website/docs/documentationfolder.Make sure you choose the correct class
estimators/transformersand namespace.DocTablepoints to correct API link.yarn run startto make sure the website renders correctly.<!--pytest-codeblocks:cont-->before each python code blocks to enable auto-tests for python samples.WebsiteSamplesTestsjob pass in the pipeline.