Nixtla / mlforecast

Scalable machine 🤖 learning for time series forecasting.
https://nixtlaverse.nixtla.io/mlforecast
Apache License 2.0
860 stars 84 forks source link

Improve error message "Found missing inputs in X_df" #273

Closed MarcoGorelli closed 10 months ago

MarcoGorelli commented 10 months ago

Description

Checklist:

Currently, the error message shows

" \"Found missing inputs in X_df. \"\n", " \"It should have one row per id and date for the complete forecasting horizon

It doesn't show what the expected number of rows is - this would've been really helpful to me yesterday when I was debugging a pipeline

With this PR, it will show:

image

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

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

CLAassistant commented 10 months ago

CLA assistant check
All committers have signed the CLA.

MarcoGorelli commented 10 months ago

thanks for your review and suggestion!

jmoralez commented 10 months ago

Can you please include the ones in the line above as well? I think those are more likely to be thousands