-
Notifications
You must be signed in to change notification settings - Fork 11
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
[JOSS] Examples #8
Comments
This is a bonus and not required at all for JOSS, but would it be possible to add a plotting output for the analysis script? molearn/examples/analysis_example.py Lines 50 to 58 in 070520c
The "generating error landscape" phase takes some time so the suspense runs high and it's a bit deceiving not to get any output. =) |
When running
Printing the shape of |
I apologise for the delayed reply. commit ceff082 should add commit 9ccea59 should add some simple plots. I agree it's a bit disappointing to run the script and get nothing at the end. commit 68f788c should fix |
Thank you @SCMusson for fixing the few issues and for adding some simple plots, everything seems to work well now. To avoid this kind of problems in the future (example scripts not following the changes in the code base), it might be worth thinking of testing them in CI (maybe only a few training step with a reduced model and data set). |
Running
analysis_example.py
in theexamples/
folder results in the following error:This happens similarly for the following paths:
molearn/examples/analysis_example.py
Lines 32 to 33 in 070520c
I'd suggest to use
os.path.join
to concatenate paths.openjournals/joss-reviews#5523
The text was updated successfully, but these errors were encountered: