-
Notifications
You must be signed in to change notification settings - Fork 394
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
cases: Model Registries #3333
cases: Model Registries #3333
Conversation
6e968b9
to
6869543
Compare
@shcheklein do you mean properly designed? That may take some time. Meanwhile we could iterate on it (here's an idea) and publish a provisional one. |
@jorgeorpinel let's iterate then, the one from the idea - you mentioned that it's more like a second image, it's bit too low level for the first one I think |
And to **package** the models, you can save them with [MLEM] in your Python | ||
training code. It automagically captures all the context needed to deploy them, | ||
and saves it to the repo. It can save your model files in cloud storage (by | ||
itself or with DVC), list and transfer them among locations, or even serve them |
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.
I still miss more clear indication that MLEM helps you use your model downstream. Pack it, deploy it. But since this is the page about MR, maybe that's ok.
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.
Yeah this paragraph improves with my understanding of MLEM. Reviewing mlem.ai/doc contents has helped 😋 . I rephrased it again, PTAL @aguschin .
This comment was marked as resolved.
This comment was marked as resolved.
@jorgeorpinel good, but looks too complicated now ... can we simplify it significantly (less arrows, may be less icons here and there, etc?) |
I rearranged things a little both with the figure (much simpler now) and benefits list + Added some links and admons (d6d01c6). Please the latest https://dvcorg-casesmodelregistry.gtsb.io/doc/use-cases/model-registry |
@jorgeorpinel it looks great! way simpler, I think we are ready to merge and then ask folks to re-draw it if needed. |
Good stuff 🎉 |
Also should we link to/from https://iterative.ai/model-registry ? I just found out about that page! |
Per #2544
Review app page: https://dvcorg-casesmodelregistry.gtsb.io/doc/use-cases/model-registry