search

E3SM-Project / zppy

E3SM post-processing toolchain
BSD 3-Clause "New" or "Revised" License
6 stars 15 forks source link

zppy-interfaces refactor #642

Closed forsyth2 closed 6 days ago

forsyth2 commented 3 weeks ago

Issue resolution

xylar commented 3 weeks ago

Yep, so far, I like the changes you're proposing!

chengzhuzhang commented 6 days ago
  • e3sm_diags/lat_lon_conus.cfg didn't appear to be used anywhere, so I created zppy/examples to house that.

@forsyth2 this .cfg file is actually referenced here: https://github.com/E3SM-Project/zppy/blob/408e9ade87db8d0d5322c01186564df1392282be/docs/source/post.rrm_simulation.cfg#L59C1-L62C39

forsyth2 commented 6 days ago

@chengzhuzhang Ah thank you. Ok, so this should actually be included under inclusions then as a possible cfg a user can use?

forsyth2 commented 6 days ago

@chengzhuzhang The commit https://github.com/E3SM-Project/zppy/pull/642/commits/be06191d3de8e956476c771da5e5c00f0dbf76a8 addresses your comment.