-
Notifications
You must be signed in to change notification settings - Fork 213
Custom CI test #230
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
Comments
#202 this is a good example written by @OliverSchacht |
This is a PR. I wasn't planning on modifying the package code but just wanted to test some things out. Is there no way to do that in the python version. |
Yeah I understand that. In order to use a custom CI test, you need to wrap it as a class in cit.py, and then directly call it by changing the parameter 'indep_test' in PC. That PR could be helpful for doing that. And you are right, there is currently no way to do it without any modification of the code. |
Thanks. Native support for custom CI tests via a function call would be a great feature. |
Thanks for the great suggestion. The CI test classes in cit.py just take X, Y and condition set as input and output the p-value, for example: causal-learn/causallearn/utils/cit.py Line 148 in e4b5f12
So, including a native support for that won't be very difficult. We will let you know when this is available. If you would like to use custom tests in the near future, the way mentioned above shouldn't be too complicated either, and I would always be happy to help if you have any questions. |
How can I implement a custom CI test to use with PC. It seems that only built in tests are supported now.
The text was updated successfully, but these errors were encountered: