Skip to content
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

Merged
merged 7 commits into from
Jul 25, 2024

Conversation

zyhu-hu
Copy link
Collaborator

@zyhu-hu zyhu-hu commented Jul 23, 2024

No description provided.

akshaysubr and others added 6 commits July 23, 2024 13:45
…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
@zyhu-hu zyhu-hu requested a review from jbusecke July 23, 2024 18:06
@codecov-commenter
Copy link

codecov-commenter commented Jul 23, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 4.57516% with 146 lines in your changes missing coverage. Please review.

Project coverage is 6.41%. Comparing base (b35af31) to head (73948c1).
Report is 25 commits behind head on main.

Files Patch % Lines
climsim_utils/data_utils.py 4.57% 146 Missing ⚠️

❗ 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.
📢 Have feedback on the report? Share it here.

@jerrylin96 jerrylin96 marked this pull request as ready for review July 23, 2024 20:38
Copy link
Collaborator

@jbusecke jbusecke left a 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.

@jbusecke jbusecke merged commit 843e1b3 into leap-stc:main Jul 25, 2024
1 check passed
jbusecke added a commit that referenced this pull request Jul 26, 2024
#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).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants