scout
scout copied to clipboard
Defaults for ecm_prep.py error based on regions
As with #273 a clean v0.8 environment will error.

Again, I would expected that the default settings for ecm_prep.py should be sufficient.
@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.
@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.
@aspeake plans on updating the --alt_regions default to EMM in a future PR