Fix: make sure that precip_models variable has the correct length

[RubenImhoff]

Fix to ensure that the _check_inputs function in /blending/steps.py looks for the correct precip_models length when a list of timesteps instead of an integer is provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.82%. Comparing base (3121cbf) to head (64c8250).
Report is 7 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #368      +/-   ##
==========================================
+ Coverage   83.50%   83.82%   +0.31%     
==========================================
  Files         159      160       +1     
  Lines       12564    12788     +224     
==========================================
+ Hits        10492    10719     +227     
+ Misses       2072     2069       -3     

Flag	Coverage Δ
unit_tests	83.82% <100.00%> (+0.31%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dnerini]

Looking good!

[ladc]

Thanks! I haven't used custom timesteps so far, so just to check -I thought it was allowed to have non-integer timesteps if you specify custom ones, so what happens in that case? (maybe requires a ceil?)

[RubenImhoff]

That is a good point, @ladc! I have not tested that (yet), but I can imagine that a ceil would solve that potential issue. Should we make sure that the number gets rounded up in that case to ensure that the non-integer timestep always falls within the total forecast length?

[dnerini]

should this be added to the hackathon?