-
Notifications
You must be signed in to change notification settings - Fork 19
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
CCD not working #105
Comments
@KyuriP Same issue here. Did you come up with any solution? I am failing to even run it in the departmental cluster which has linux system. I am personally using MAC. |
@KyuriP I was just talking with @roys8001 about this. I had taken CCD out of the interface because of inaccuracy, not because it doesn't function, but to be fair if there's interest I'll put it back in. It may be buggy in the version that's in rcausal, which is very old, though:
I'm trying to set up a new way of running Tetrad algorithms in R that goes through Python, which I'm hoping will be more stable. The setup is perhaps more annoying, but I've given it here for a Mac. This is new as of yesterday and doesn't include CCD yet, but if you want to test out the install procedure and give me feedback, I would be most grateful. https://github.com/cmu-phil/py-tetrad/blob/main/pytetrad/R/REAMDE.md I was going to add some tests for discrete data today and add CCD (at @roys8001 's request). |
I just put CCD back in the Tetrad interface (on my local branch of the repo) and tested it on some discrete data and it worked. So I don't think discrete data is throwing it off. Let me add it to the py-tetrad R gadget and see if I can get it to work with some discrete data. I'm working on this currently so it should be available later today. |
CCD is now available for continuous data in py-tetrad for R and is working for my example. |
By the way, the CCD output graph has underlinings, which I'll have to find a way to print in R (maybe, a method, 'print_underlinings'). Also, Thomas never worked out knowledge for the CCD algorithm, so if you set any knowledge before running CCD, it will complain. |
We have some offers to test the instructions for this on our end as well, in case you want to wait and see if they change. Also, maybe the whole setup could be done automatically in a Docker container in VSC--we'll look into it. |
Sure. But I shall try to run it too and drop in some queries if I got struck at some step. |
Great! Thank you so much! |
@roys8001 Were you able to get https://github.com/cmu-phil/py-tetrad/tree/main/pytetrad/R installed, and were you able to use CCD with it? |
Hi Dr Ramsey, something urgent came up. I shall try again with this from Friday. |
It's no rush :) |
I am trying to run CCD but it throws me an error below.
The text was updated successfully, but these errors were encountered: