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

add MLflow logging example #892

Closed cargecla1 closed 5 months ago

cargecla1 commented 7 months ago

Addressing MLflow code example to integrate easily Fixes #825

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

CLAassistant commented 7 months ago

CLA assistant check
All committers have signed the CLA.

cla-ra3426 commented 7 months ago

Hello @jmoralez , Do you have time to look at this?

Cheers!

cargecla1 commented 6 months ago

Hello @jmoralez ,

I just included all suggestions except for a suggestion to remove the #Registering model code section. I explained why on my comment to resolve that suggestion.

Thank you for all these suggestion! :)

jmoralez commented 5 months ago

Hey @cargecla1, I just pushed some changes. Please let me know if they look good to you, if they do we can merge this.

cargecla1 commented 5 months ago

Hey @cargecla1, I just pushed some changes. Please let me know if they look good to you, if they do we can merge this.

Hello @jmoralez ,

These changes look great, thank you for these additions! I believe this is now ready to merge.

Thank you very much for your support!

jmoralez commented 5 months ago

Thanks a lot for the contribution!