Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Model evaluation documentation asserts that datasetsforecast is required, but it isn't #833

Closed
quant5 opened this issue Apr 30, 2024 · 2 comments

Comments

@quant5
Copy link

quant5 commented Apr 30, 2024

Description

In many of the "Model Evaluation" sections within the statsforecast docs, it's mentioned that datasetsforecast is a required installation in order to get access to error metrics.

However, this is not the case, as one can simply import them from utilsforecast, which comes installed with statsforecast.

from utilsforecast.losses import *

My sense is that the docs would read cleaner without this additional unnecessary external install requirement. Let me know what you think.

Link

https://nixtlaverse.nixtla.io/statsforecast/docs/models/autoarima.html#model-evaluation
https://nixtlaverse.nixtla.io/statsforecast/docs/models/optimizedtheta.html#model-evaluation

And others.

@jmoralez
Copy link
Member

I think this is closely related to #834 and would be fixed by it, should we close this one?

@quant5
Copy link
Author

quant5 commented May 8, 2024

Sure, will close this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants