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

removed documentation for missing argument #913

Closed yarnabrina closed 7 months ago

yarnabrina commented 7 months ago

Closes #910

Documentation for LSTM mentioned encoder_activation argument, but it was not present in __init__. This PR removes this from the docstring.

CLAassistant commented 7 months ago

CLA assistant check
All committers have signed the CLA.

jmoralez commented 7 months ago

Thanks @yarnabrina. Can you please also remove it from the notebook? You can refer to the contributing file for more details. Also please let us know if you feel something's missing from it.

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

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB