scout icon indicating copy to clipboard operation
scout copied to clipboard

Defaults for ecm_prep.py error based on regions

Open dewittpe opened this issue 3 years ago • 2 comments

As with #273 a clean v0.8 environment will error.

image

Again, I would expected that the default settings for ecm_prep.py should be sufficient.

dewittpe avatar Sep 08 '22 17:09 dewittpe

@dewittpe this is expected behavior, though as you note not necessarily intuitive. Ultimately we need to overhaul the climate_zone input to be more general (e.g., rename to region) since it's no longer just AIA climate zones; we also should probably switch the default region settings to EMM, which would avoid this error. However, those changes will take longer to implement and verify.

For now, we have this error message to inform users of what to do, and I will also add some guidance to the Quick Start Guide about running ecm_prep with EMM region settings. Let's leave this open until we have a more permanent solution.

jtlangevin avatar Oct 12 '22 10:10 jtlangevin

@dewittpe this is expected behavior, though as you note not necessarily intuitive.

:rage:

We can take the conversation off-line. I would very much like to know what went into the decision to release a software package knowing that end users would run into an error when using the documented default settings.

dewittpe avatar Oct 12 '22 15:10 dewittpe

@aspeake plans on updating the --alt_regions default to EMM in a future PR

aspeake avatar Jan 24 '24 22:01 aspeake