-
Notifications
You must be signed in to change notification settings - Fork 343
compare_paramfiles tool #3622
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
compare_paramfiles tool #3622
Conversation
Only basic testing of argument parsing and checking.
…ch between DataArrays.
slevis-lmwg
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.
@samsrabin thank you for this impressive PR! The compare_paramfiles tool is much more informative than cprnc. Your implementation looks very well thought out and you put together a nice list of unit and sys tests.
@samsrabin and I went over this in detail and @samsrabin took notes.
Description of changes
Adds a utility to compare parameter files. Compares both "raw" values and values after masking and scaling (applying
_FillValueto convert missing values to NaN, multiplying byscale_factor, and addingadd_offset).Example usage; results truncated:
Specific notes
Contributors other than yourself, if any: None
CTSM Issues Fixed (include github issue #): None
Are answers expected to change (and if so in what way)? No
Any User Interface Changes (namelist or namelist defaults changes)? No
Does this create a need to change or add documentation? Did you do so? Yes; yes.
Testing performed, if any: All unit and system tests pass. Results on real paramfiles look good.
clm_pymodstest suite: Passed at/glade/derecho/scratch/samrabin/tests_1219-073947de