You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While we primarily want to support standardized regions for analysis, it may be useful to researchers to be able to define their own regions for ad-hoc analyses (e.g., diagnosing issues in a certain region in a high-resolution model).
The primary complication I foresee with this is that regions is specified as a list of strings that default_regions.py then maps to actual regions, so the question is how to specify a region in a cfg. Perhaps it could be parsed from a string (e.g., latitude=(0.0, 45.0, "ccb"), longitude=(210.0, 310.0, "ccb") => "0.0,45.0,ccb,210.0,310.0,ccb")? At that point, it seems like another parameter would have to be introduced, something like user_defined_regions=.
Is your feature request related to a problem?
While we primarily want to support standardized regions for analysis, it may be useful to researchers to be able to define their own regions for ad-hoc analyses (e.g., diagnosing issues in a certain region in a high-resolution model).
Describe the solution you'd like
Add the ability for users to specify user-defined regions in the
cfg
rather than needing to actually change thee3sm_diags
source code in https://github.com/E3SM-Project/e3sm_diags/blob/main/e3sm_diags/derivations/default_regions.py.The primary complication I foresee with this is that
regions
is specified as a list of strings thatdefault_regions.py
then maps to actual regions, so the question is how to specify a region in a cfg. Perhaps it could be parsed from a string (e.g.,latitude=(0.0, 45.0, "ccb"), longitude=(210.0, 310.0, "ccb")
=>"0.0,45.0,ccb,210.0,310.0,ccb"
)? At that point, it seems like another parameter would have to be introduced, something likeuser_defined_regions=
.Describe alternatives you've considered
e3sm_diags
to support a new region. This risks bloating the set of standard regions.Additional context
No response
The text was updated successfully, but these errors were encountered: