Skip to content

1380 Colorbar fix for mean_climate difference plots #1388

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.

Sign up for GitHub

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

Merged
lee1043 merged 2 commits into from
Jun 3, 2026
+12 −4
Merged

1380 Colorbar fix for mean_climate difference plots #1388

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c246729
changed colorbar extension to both
jsg5776 Jun 3, 2026
38a4062
clarifying comments
jsg5776 Jun 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 12 additions & 4 deletions pcmdi_metrics/mean_climate/lib/plot_clim_maps.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1176,6 +1176,14 @@ def _load_variable_setting(

# Check if the variable and level exist in the settings
if data_var in var_setting_dict:
# The variable might exist, but not the level. In that case, use the colormaps for the first level within the variable dictionary
first_key = next(iter(var_setting_dict[data_var]))
cmap = _get_colormap(
var_setting_dict[data_var][first_key].get("colormap", None)
)
cmap_diff = _get_colormap(
var_setting_dict[data_var][first_key].get("colormap_diff", None)
)
if level in var_setting_dict[data_var]:
settings = var_setting_dict[data_var][level]
contour_levels = settings.get("contour_levels", None)
Expand Down Expand Up @@ -1211,10 +1219,10 @@ def _load_variable_setting(
cmap = plt.get_cmap("jet")
if cmap_diff is None:
cmap_diff = plt.get_cmap("RdBu_r")
if cmap_ext is None: # Prevent colorbar extension
cmap_ext = None
if cmap_ext_diff is None: # Prevent colorbar extension
cmap_ext_diff = None
if cmap_ext is None: # Allow colorbar extension
cmap_ext = "both"
if cmap_ext_diff is None: # Allow colorbar extension
cmap_ext_diff = "both"

if diff:
return contour_levels_diff, cmap_diff, cmap_ext_diff
Expand Down
Loading