-
Notifications
You must be signed in to change notification settings - Fork 40
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
adding online testing codes/notebooks #98
adding online testing codes/notebooks #98
Conversation
…ference routines for it add my own notebook directory rm ignore hu_notebooks ignore notebooks add Hu_models directory with a MLP folder add mlp training script debug moving conf into training folder debug debug debug debug debug debug debug add dyn norm stats Squash all downstream changes update modulus info in downstream overview minor update on downstream overview delete unuseful argumens in config_single.yaml minor update on create_dataset add the the missing climsim_datapip.py; fix input filename consistency for create_dataset notbook remove an outdated argument in training scripts
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #98 +/- ##
========================================
- Coverage 6.72% 6.41% -0.32%
========================================
Files 1 1
Lines 847 982 +135
========================================
+ Hits 57 63 +6
- Misses 790 919 +129 ☔ View full report in Codecov by Sentry. |
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.
@zyhu-hu has tested jupyter book locally so Ill merge this now. Please double check results once the site is redeployed.
#98 is not showing up properly. This is due to the subtlety that we want to have md files show up in the readme but also the book (and thus we have to copy files from root to the website dir at build time). Thanks @zyhu-hu for catching this. If we expect more folks to contribute, we should document this better (and potentially implement a website preview).
No description provided.