-
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
Unable to specify random seed for sampling in Bootstrapping? #103
Comments
@yasu-sh Thanks for reporting the bug. I'll take a look at it. This should not happen. If it's not working in causal-cmd then it will not work in tetrad, too. I'll get back to you on this. |
@yasu-sh I'm just curious: What are the reasons you can't move to 17? I'm just curious. We have people who want us even to move to 21 and stay up-to-date with Java versions... it's so hard to find a good version... |
@jdramsey It's natural for you to have the quesion. My idea is as below: Let me know if you notice this idea does not work. |
@kvb2univpitt @jdramsey I am appreciated if you focus on confirming the results be reproductive. It is important. |
@yasu-sh We decided to stick with Java 17 and move to Java 21 in the future. The latest version of Java provides better security and performances. We take advantages of some of the new features in each Java release for better performances and code readability. For an example, we replaced some Java Classes with Java Records for specific use. We are planning to use virtual threads instead of platform thread in the future when we switch over to Java 21 LTS If your case, you can download and extract the zip or tar file version of OpenJDK17 to your home directory. Instead of typing |
@kvb2univpitt Thanks for telling on the future direction. It is complicated since causal-cmd is called by R. |
@kvb2univpitt @jdramsey I have found a workaround for using tetrad 1.12 on Java 17(Corretto 17 zip-extracted on windows) at our server. If you need more information, let me know. |
@yasu-sh I'll coordinate with Kevin on this. It's possible I broke something, but I'm not sure how to tell quite yet. |
@kvb2univpitt @jdramsey Thanks for your re-confirmation on Tetrad also. I might create another issue on Tetrad. |
@yasu-sh Kevin and I looked at this yesterday (I think). it's a problem in 7.6.1 but not in the development branch. Hmm. |
By the way I believe we're almost ready to publish a new version of Tetrad. I'll double check to make sure the bootstrapping issue works in it. |
@jdramsey Thanks a lot! I hope it was just a version-specific issue. |
Dear developers, @jdramsey, @kvb2univpitt
Several months ago I made a request on the bootstrapping in:
#80
I am grateful for the added functionality.
But recently I have noticed that even when setting a seed and executing with bootstrapping, I cannot obtain reproducible results in causal-cmd v. 1.11.
I attached two log files, both analyzing the same data with the same parameters.
If there are any mistakes how to set seed or other parameters, please let me know.
If not, we would be appreciated if you could check whether the code generates randomness under the seed setting.
Hopefully the latest version(causal-cmd v.1.12 or above) is available on Java 11 since our environment is limited.
(if you updated the version from 8 to 17 and skipped Java 11 since there were no particular reasons for Java 17.
tetrad_240308-111141.log
tetrad_240308-111250.log
The text was updated successfully, but these errors were encountered: