Fix the standard deviation in the sea ice contour plots.#403
Open
dabail10 wants to merge 3 commits into
Open
Conversation
mnlevy1981
approved these changes
Apr 20, 2026
Collaborator
mnlevy1981
left a comment
There was a problem hiding this comment.
Looks good! I know np.nanstd() fixed the issue with vmin and vmax, and I like how the arguments were cleaned up (listing vmin before vmax and passing [constant]*field_std instead of "field_std*[constant])
Collaborator
Author
|
@duvivier Can you take a quick look at this? |
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
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.
Description of changes:
This addresses issue #402. Turns out that I needed nanstd. Also tweak the colormaps and mask.
All PRs Checklist:
pre-commitchecks passed (#8 in Adding Notebooks Guide)?New notebook PR Additional Checklist (if these do not apply, feel free to remove this section):
parameters? These can cause confusing warnings that show up asDAG build with warnings./glade/campaign/cesm/development/cross-wg/diagnostic_framework/CUPiD_obs_dataand ensured that it follows this format within that directory:COMPONENT/analysis_datasets/RESOLUTION/PROCESSED_FIELD_TYPE?edit [MNL]: removed superfluous checkbox from description section