-
Notifications
You must be signed in to change notification settings - Fork 13
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
Pairing with mlflow release api definition #59
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #59 +/- ##
===========================================
+ Coverage 78.07% 89.18% +11.11%
===========================================
Files 10 19 +9
Lines 333 333
===========================================
+ Hits 260 297 +37
+ Misses 73 36 -37 ☔ View full report in Codecov by Sentry. |
7522838
to
50004ce
Compare
Changing |
@pebeto Thanks for what looks like a considerable effort 🎉 Unfortunately, I don't really have the bandwidth to review PRs at MLFlowClient, and was never really that familiar with this side of things. (Still happy to review at MLJFlow.jl). Perhaps see if one of the other recent contributors would care to review. I see there is still a to-do list. Perhaps you want to flag this as a "Draft", unless you are seeking a review already? |
Are there any updates to this PR? This is helpful |
0a70170
to
99b8eb8
Compare
ccd597a
to
1333338
Compare
I'm merging the changes, but continuing analyzing an optimal solution for artifact operations in #55 . |
This PR contains a major update in the project.
mlflow
REST API.mlflow
latest release.Important
This update comes with some breaking changes. Projects like MLJFlow.jl will need to update carefully. It will be the same with personal projects.
TODO list:
RegisteredModel
andModelVersion
operations.RegisteredModel
ModelVersion
mlflow
authentication REST API operations.User
operationsExperimentPermission
operationsRegisteredModelPermission
operationsUseful links: