MLflow for Darts implementation - #3022
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
Hey @daidahao, adding this draft PR in the meantime so you and @dennisbader can have a look at what I have currently regarding the integration. There are still some decisions I am not too thrilled about and decisions to be made about the overall direction, but I'm happy to talk more about it during the meeting. Thanks for being so active for the library, really nice to be working together :) |
There was a problem hiding this comment.
Really nice improvements, thanks a lot @jakubchlapek. I've played around a lot with edge cases and allmost all of them looked great :) Kudos to that!
- I've added a couple more suggestions from these left over edge-cases.
- Make sure to check the code coverage report and fix any missing parts (if applicable)
After those changes have been applied, we should do a final round of going through the docs and the example notebook to make sure that everything is correct and easily understandable by the users :)
Checklist before merging this PR:
Addresses #2092 .
Summary
Provides a custom MLflow flavor for Darts on Darts' side. Supports autologging, logging, saving and loading of the models.
This PR focuses on the base MLflow integration, leaving serving of the models to be discussed in the future.
Included an example quickstart for the integration, however consider all of this a draft :)
Find example code in the .ipynb, however also providing a code snippet here as a quick reproducible example: