-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix & document inference/training step #92
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
It would make sense to maybe rename the "testing" directory into "inference", as otherwise people might expect to find software tests in there. |
@arthurBarthe , in the Jupyter Notebook there are a lot of sections now labelled "IGNORE THIS". Can we delete these, or what are they used for and why should they be ignored? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy now after changes have been implemented
I am doing this renaming |
The goal of this PR is to fix/clean up the inference code (the testing step), and add documentation regarding its use.
- [ ] describe how to load the inference output data from the notebook test_global_control-> This will be done in Notebooks cleanup #82