-
Notifications
You must be signed in to change notification settings - Fork 40
Support ISMIP6 Forcing Generation for Greenland #853
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
base: main
Are you sure you want to change the base?
Conversation
1af8739 to
deb7587
Compare
Ensure's the climatology can be found by multiple steps, in order process the SMB anomoly files provided by ISMIP6.
- Now packing atm/ocn forcing vars into their own respective forcing files - moved shared utilites to common file - fixed bug in returning conctenated atm files from file finder
deb7587 to
08c38df
Compare
Made all variables snake case. Add docstings to all functions, methods, and classes.
|
Based on offline discussion, I don't actually think this needs a MALI PR. The elevation dependent components of the the forcing (i.e. |
| # need a special way to treat the RACMO datasets | ||
| if expr_name in ["ctrl", "hist"]: | ||
| continue |
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.
@trhille @matthewhoffman Now I'm remembering what stalled this PR originally.
What should we do about atm/ocn data during a historical run? Should we do a different historical run for each GCM? I think I recall ISMIP6 cautioning against that.
https://theghub.org/groups/ismip6/wiki/ISMIP6-Projections-Greenland
The unforced control run should be easy to support.
This PR aims to enable ISMIP6 forcing file generation for the Greenland Ice Sheet.
Checklist
api.rst) has any new or modified class, method and/or functions listedE3SM-Projectsubmodule has been updated with relevant E3SM changesMALI-Devsubmodule has been updated with relevant MALI changesTestingin this PR) any testing that was used to verify the changes