Skip to content

ENSO portrait plot color code update in notebook#1377

Merged
lee1043 merged 2 commits into
mainfrom
enso_fix_check
May 24, 2026
Merged

ENSO portrait plot color code update in notebook#1377
lee1043 merged 2 commits into
mainfrom
enso_fix_check

Conversation

@lee1043

@lee1043 lee1043 commented May 21, 2026

Copy link
Copy Markdown
Contributor

Testing #1372 and document its outcome

@lee1043 lee1043 marked this pull request as ready for review May 21, 2026 21:19
@lee1043

lee1043 commented May 21, 2026

Copy link
Copy Markdown
Contributor Author

Hi @zhangshixuan1987, could you please take a look the following notebook to see the change is consistent to that you expected?

https://github.com/PCMDI/pcmdi_metrics/blob/71db16aac31cbb2f62d1f1ccb70875df0ecd510f/pcmdi_metrics/enso/lib/summary_plot_lib/enso_portrait_plot.ipynb

@zhangshixuan1987

Copy link
Copy Markdown
Collaborator

@lee1043 Hi Jiwoo, the results look consistent with what I expected. I wanted to ask whether the color scale looks good to you, or if you think we should adjust it to improve the visualization.

@lee1043 lee1043 merged commit be94551 into main May 24, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants