-
Notifications
You must be signed in to change notification settings - Fork 923
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add examples from "Causal Inference and Discovery in Python" to tests #1165
Add examples from "Causal Inference and Discovery in Python" to tests #1165
Conversation
3e15831
to
518ba93
Compare
8f93548
to
fbc80d8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR @rahulbshrestha . Apologies it took me longer than I thought to review this.
The PR looks great. I just added a couple of minor comments. Once you address them, I'm happy to merge.
tests/causal-inference-discovery-book/test_causal_inference_discovery_book.py
Show resolved
Hide resolved
tests/causal-inference-discovery-book/test_causal_inference_discovery_book.py
Outdated
Show resolved
Hide resolved
tests/causal-inference-discovery-book/test_causal_inference_discovery_book.py
Outdated
Show resolved
Hide resolved
tests/causal-inference-discovery-book/test_causal_inference_discovery_book.py
Outdated
Show resolved
Hide resolved
db8a1ee
to
8eb290a
Compare
Thanks for the edits, @rahulbshrestha There is a build error due to config issue with econml tests. I will resolve that and merge this PR after that. |
Signed-off-by: Rahul Shrestha <rahulshrestha0101@gmail.com> added chapter 6 Fix Signed-off-by: Rahul Shrestha <rahulshrestha0101@gmail.com> Add chapter 8 Signed-off-by: Rahul Shrestha <rahulshrestha0101@gmail.com> Fix formatting issue with linter Add references
Signed-off-by: rahulbshrestha <rahulshrestha0101@gmail.com>
Signed-off-by: rahulbshrestha <rahulshrestha0101@gmail.com>
d04360b
to
ec6351c
Compare
Signed-off-by: Amit Sharma <amit_sharma@live.com>
Thanks you @rahulbshrestha for adding these tests. sorry for the delay in fixing the econml dependencies! |
This PR intends to add tests from the book "Causal Inference and Discovery In Python" (link) in order to ensure backwards compatibility to future releases of doWhy
Related issue: Add examples from "Causal Inference and Discovery in Python" to the tests to ensure backwards compatibility #970