fix: fitted nonrecursive cv with horizon >= 10

Nixtla / mlforecast

Scalable machine 🤖 learning for time series forecasting.

https://nixtlaverse.nixtla.io/mlforecast

Apache License 2.0

789 stars 74 forks source link

fix: fitted nonrecursive cv with horizon >= 10 #332

Closed adriaanvh1 closed 3 months ago

adriaanvh1 commented 3 months ago

Description

Fixes #331 A test is added that passes with the fix, and fails without. The bug is due to a faulty regex that filters column names with only one digit for the target.

Checklist:

[x] This PR has a meaningful title and a clear description.
[x] The tests pass.
[x] All linting tasks pass.
[x] The notebooks are clean.

review-notebook-app[bot] commented 3 months ago

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

CLAassistant commented 3 months ago

All committers have signed the CLA.