-
Notifications
You must be signed in to change notification settings - Fork 19
Translate variable names for Omega before baseline validation #398
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
Translate variable names for Omega before baseline validation #398
Conversation
TestingAfter struggling to test on Perlmutter, I switched to Chrysalis (with Intel), where I was able to successfully run both the |
|
@cbegeman, could you give this a quick look when you can? I don't think more testing is needed than what I already did. |
cbegeman
left a comment
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.
I really like this change! My only question would be whether we should do the component is ocean check in ocean_io_step's __init__ rather than its methods.
I'll look into that but I think probably so. |
|
There seems to be an easier fix that I'll push in just a second. |
This merge removes unneeded Omega --> MPAS-Ocean dictionaries. It also adds methods for translating a list of variable names, rather than a dataset (useful for valiation variables). This approach will prevent redundant building of the translation maps for each step that needs them and will make the translation available beyond OceanIOSteps
...in OceanIoStep and OceanModelStep
6380088 to
e221c6e
Compare
|
@xylar Yes, that looks great. |
To accomplish this, this PR moves Omega <--> MPAS-Ocean variable translation to the Ocean component class. It removes the unneeded Omega --> MPAS-Ocean dictionaries (we can use the MPAS-Ocean --> Omega dictionaries for both direction). It also adds methods for translating a list of variable names, rather than a dataset (useful for validation variables).
This approach prevents redundant building of the translation maps for each step that needs them and makes the translation available beyond
OceanIOSteps.OceanIOStepshas been updated to use the methods fromOceancomponent for mapping variable and dimension names.Checklist
api.md) has any new or modified class, method and/or functions listedTestingcomment in the PR documents testing used to verify the changesFixes #395