-
Notifications
You must be signed in to change notification settings - Fork 32
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
Mixed phase partition LCF diagram #675
Conversation
@zyuying and @susburrows I have implemented codes for the T5050 diagram, here is the link to the example figure. The unresolved part is the variable change for CLDICE from v2 to v3 P3. I will update this PR as soon as the new variable name/convention is decided on. |
This looks great! It will be great to have this diagnostic available in e3sm_diags so we can easily check on global cloud phase partitioning during model development. |
@zyuying I was wondering is there an update on the variables to be used for this diagnostics? I'm hoping to have this branch finalized and merged by the end of this month for a new e3sm_diags release. |
Based on a discussion with @zyuying , this PR can be merged to prepare for joining the e3sm-unified release. There will be a separate PR to add simulator comparison and to accommodate v3 output. |
New feature requested by #653