Updates to quickstat plotting code #130
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ce zonal mean (latitude vs height) comparison plots when the compared experiments have different levels or different forecast lengths or different time increments. Fixes for corcmp routines were to have EXP interpolate to CTL levels to define differences for zonal comparisons, look forshortest forecast length among all files, find the largest common pressure level to define as toplev, look for one time increment being 2x the other and adjust dimensions accordingly, and change the naming convention when the NDAY environment variable is set so that these plots are not seen as a different month by movestat. Fixes for syscmp routines were to identify the experiment with largest time increment and adjust dimension accordingly, find the largest common pressure level to define as toplev, and allow for limiting dimensions (fewer levels, shorter forecast length, etc.) to come from CTL or EXP.
michellefrazer
added
0 diff trivial
|
Well this is complex! Good job figuring this out. |
… model and control (instead of difference for 95% t-test confidence level)
sdrabenh
approved these changes
Jun 26, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Code was updated to allow quickstat jobs (corcmp and syscmp) to produce zonal mean (latitude vs height) comparison plots when the compared experiments have different levels or different forecast lengths or different time increments. Fixes for corcmp routines were to have EXP interpolate to CTL levels to define differences for zonal comparisons, look for shortest forecast length among all files, find the largest common pressure level to define as toplev, look for one time increment being 2x the other and adjust dimensions accordingly, and change the naming convention when the NDAY environment variable is set so that these plots are not seen as a different month by movestat. Fixes for syscmp routines were to identify the experiment with largest time increment and adjust dimension accordingly, find the largest common pressure level to define as toplev, and allow for limiting dimensions (fewer levels, shorter forecast length, etc.) to come from CTL or EXP.