ensure lags are positive integers

Nixtla / mlforecast

Scalable machine 🤖 learning for time series forecasting.

https://nixtlaverse.nixtla.io/mlforecast

Apache License 2.0

841 stars 80 forks source link

ensure lags are positive integers #232

Closed jmoralez closed 11 months ago

jmoralez commented 11 months ago

Description

The lags that are used in the lags and the keys of the lag_transforms arguments must be positive integers, this adds checks to enforce that and avoid unexpected behavior.

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 11 months ago

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.