Nixtla / neuralforecast

Scalable and user friendly neural :brain: forecasting algorithms.
https://nixtlaverse.nixtla.io/neuralforecast
Apache License 2.0
2.75k stars 316 forks source link

AutoLSTM.__init__() got an unexpected keyword argument 'callbacks' #922

Open pranavvp16 opened 4 months ago

pranavvp16 commented 4 months ago

What happened + What you expected to happen

AutoLSTM class gives unexpected keyword argument error when passes with parameter callbacks. It should accept the parameter as mentioned in the docs

Versions / Dependencies

neuralforecast==1.9.4

Reproduction script

from neuralforecast.auto import AutoLSTM
model = AutoLSTM(num_samples = 10, callbacks = None)

Issue Severity

High: It blocks me from completing my task.

JQGoh commented 4 months ago

@pranavvp16

May I check what is the version of neuralforecast that you tested? Because there is no such version of 1.9.4

Using neuralforecast==1.6.4 Edit: Using a dev version of neuralforecast (beyond version 1.6.4)

from neuralforecast.auto import AutoLSTM
model = AutoLSTM(h=10, num_samples = 10, callbacks = None)

works fine without any errors. The issue with your proposed reproduction script is that it is missing the required parameter h.

jmoralez commented 4 months ago

That's only on the dev version at the moment, it'll be available in the next release. Or you can install from github

pranavvp16 commented 4 months ago

gotcha, why are docs of neuralforecast ahead of actual package. Cause I was. trying to implement this parameter after reading the docs

jmoralez commented 4 months ago

We haven't setup versioning in our new docs engine (I'm not sure if it's possible yet) and it's been a long time since the last release (~4 months). I'll leave this open as a reminder to try to setup the versioning.