-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement: multiple save and load for mlflow registry #416
Conversation
yleilawang
commented
Sep 19, 2024
- Implemented save_multiple and load_multiple for mlflow registry
- Test cases for implementation.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #416 +/- ##
==========================================
- Coverage 92.20% 92.02% -0.18%
==========================================
Files 98 98
Lines 4834 4865 +31
Branches 437 442 +5
==========================================
+ Hits 4457 4477 +20
- Misses 277 285 +8
- Partials 100 103 +3 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, update the version for pyproject.toml
@s0nicboOm we have not released any previous versions right? |
Signed-off-by: Leila Wang <[email protected]>
Signed-off-by: Leila Wang <[email protected]>
Added changes for:
|
Profiling Updates
--> 9.074686333002319 4.829918750001525 |
Signed-off-by: Leila Wang <[email protected]>
Signed-off-by: Leila Wang <[email protected]>
Signed-off-by: Leila Wang <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments.
Otherwise LGTM!
Signed-off-by: Leila Wang <[email protected]>