Closed claalmve closed 1 month ago
I am not aware of any usage of param
that’s more appropriate here. Looks like CavaParams
helps to capture inputs to cava_data
that are not intended to be supported. The introduction of param
makes input validation much more efficient👍
It’d be great if users are asked to clarify their intention if they set approach="time"
but also provided a warming level.
Description of PR
Summary of changes and related issue Including parameter validation for SCE pilot. This should make the code more readable. Additionally,
cava_data
now catches multiple logic errors at once, allowing the user to fix multiple issues at once rather than fixing them one by one.WIP: Catching multiple input validation errors at once. I.e. inputting
meannn
formetric_calc
andDynamicallll
fordownscaling_method
will only show themeannn
as an issue. Once that is addressed, then it showsDynamicallll
as an issue.Type of change
Checklist: