-
Notifications
You must be signed in to change notification settings - Fork 8
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
Causal-cmd documentation has outdated examples #87
Comments
@kvb2univpitt Do you know if this was every fixed? I'm guessing yes... |
@jdramsey No. This issue has not been fixed. The code in Tetrad is changing a lot. I'll wait until the next official release of Tetrad to update the causal-cmd to use the latest tetrad-lib. Once that's done, I'll update the documentation. If I do it now, I will have to do it again later if anything else changes. |
@jdramsey @kvb2univpitt I opened an issue on bd2kccd/bd2kccd.github.io#17 . |
Not pushed in stage directory yet.
|
Moved to the causal-cmd issues list from the Tetrad issues list.
From a causal-cmd user.
The provided example:
java -jar causal-cmd-1.0.0-jar-with-dependencies.jar --algorithm fges --data-type continuous --dataset Retention.txt --delimiter tab --score sem-bic
Works when changed to:
java -jar causal-cmd-1.3.0-jar-with-dependencies.jar --algorithm fges --data-type continuous --dataset Retention.txt --delimiter tab --score sem-bic-score --skip-latest
Because:
Error: Unable to access jarfile causal-cmd-1.0.0-jar-with-dependencies.jar
No such score 'sem-bic'.
Running version 1.3.0 but the latest version is 1.2.0. To disable checking use the skip-latest option.
Also, I was unable to understand the file containing priors based on the information given on the help page. The included Retention.prior file does not look the same as the example that is shown on the help page.
The text was updated successfully, but these errors were encountered: