Nixtla / neuralforecast

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

[FEAT] Add option to modify the default configure_optimizers() behavior #1015

Open JQGoh opened 4 months ago

JQGoh commented 4 months ago

Rationale

This shall deprecate the earlier work introduced in: https://github.com/Nixtla/neuralforecast/pull/998

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

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

jmoralez commented 4 months ago

Hey @JQGoh, thanks a lot for trying this approach! Although I think it's fine to add more arguments for now, we can give this a shot when we work towards v2.0.

JQGoh commented 4 months ago

Hey @JQGoh, thanks a lot for trying this approach! Although I think it's fine to add more arguments for now, we can give this a shot when we work towards v2.0.

In that case, I shall put this on hold then.