search

E3SM-Project / e3sm_diags

E3SM Diagnostics package
https://e3sm-project.github.io/e3sm_diags
BSD 3-Clause "New" or "Revised" License
39 stars 32 forks source link

Support monthly average and variable name convention for EAMxx #712

Closed chengzhuzhang closed 8 months ago

chengzhuzhang commented 1 year ago

Two use cases: For longer than 1 year simulation: Support both monthly average ['01', '02', ..., '12'] in viewer in addition to seasonal and annual mean.

For shorter than 1 year simulation: Support visualization the averaged snapshot.

This PR is to support the first use case, it closes #348 and partially #652. This PR also add support for variable names from EAMxx to support its output.

An example run with regridded EAMxx output is shown here. The run script for this run is linked here.

chengzhuzhang commented 1 year ago

Commit #0c74dd9 to enable 12 months means as default in core sets. I considered to add logic in season selector to support monthly means other than the original 5 (ANN, and 4 seasons), but it appears to adding these 12 month to each cfg files is less intrusive.

With this PR the default option for seasons parameter is still ["ANN", "DJF", "MAM", "JJA", "SON"], if any of the 12 months i.e. "01", "06" is present in seasons, they can be calculated can presented in viewer.

chengzhuzhang commented 1 year ago

Note to myself: remember to update code here to support monthly data.

Update: handled in https://github.com/E3SM-Project/e3sm_diags/pull/712/commits/9772f90725814a0345ac429743c894e6401e182b

chengzhuzhang commented 8 months ago

@kaizhangpnl Just for your information, this PR to support EAMxx regridded data in E3SM Diags is ready for review. I think you or someone from you team can be great early testers. I also included the result viewer in this thread for review. We can work together to refine this feature.

On the other hand, @golaz has another PR in zppy to support processing native EAMxx output. We are planning to merge shortly. I will tag you again when the PR is ready to merge after some testing...

chengzhuzhang commented 8 months ago

@tomvothecoder Tagging you for review. Changes from this PR might need to be carried to cdat migration branch at a later time..

kaizhangpnl commented 8 months ago

@kaizhangpnl Just for your information, this PR to support EAMxx regridded data in E3SM Diags is ready for review. I think you or someone from you team can be great early testers. I also included the result viewer in this thread for review. We can work together to refine this feature.

On the other hand, @golaz has another PR in zppy to support processing native EAMxx output. We are planning to merge shortly. I will tag you again when the PR is ready to merge after some testing...

Great. I'd be happy to test it. Thanks for the useful information!

chengzhuzhang commented 8 months ago

Tested it on Perlmutter and reproduced the plots.

Awesome! Thank you, Kai!