-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Description
In the latest finch updates, I forgot to update the "not implemented" indicator list :
finch/finch/processes/__init__.py
Lines 49 to 56 in 3b3deb0
| not_implemented = [ | |
| "DC", | |
| "FWI", | |
| "RH", | |
| "RH_FROMDEWPOINT", | |
| "E_SAT", | |
| "HUSS", | |
| ] |
I'm not sure what is the purpose of this list, but I can guess it was there to avoid non-resampling indicator to be wrapped into finch?
First question : Why not?
Second question : Taking ensemble statistics on daily variables doesn't make much sense, no? We could add those "converters" to finch, but avoid creating ensemble statistics processes from them?
Currently, the ensemble problem only concerns corn_heat_units, all other non-resampling indicators accept variables that BCCAQv2 doesn't provide. But that's just random luck.
Metadata
Metadata
Assignees
Labels
No labels