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
I think we could simply map it to xclim's STRING ? Unless there's a simple way to detect if the allowed values' type, my guess is that we don't really need Literals for anything else than strings. So here :
And if we do that, maybe we could apply the same for all op and method arguments throughout the library ? And maybe use the annotation to parse the choices property of xclim.core.indicator.Parameter ?
I think we could simply map it to xclim's STRING ? Unless there's a simple way to detect if the allowed values' type, my guess is that we don't really need Literals for anything else than strings. So here :
xclim/xclim/core/utils.py
Lines 659 to 660 in 98c251c
And if we do that, maybe we could apply the same for all
op
andmethod
arguments throughout the library ? And maybe use the annotation to parse thechoices
property ofxclim.core.indicator.Parameter
?Originally posted by @aulemahal in #1796 (comment)
The text was updated successfully, but these errors were encountered: