-
Notifications
You must be signed in to change notification settings - Fork 12
GS: reorg / rewrite #58
Comments
The checked boxes are being addressed in #188.
Not doing this, don't think it's needed in the current structure. We'll address this (in the other way though) once we start working on #190
Almost done, except for |
Ok, the only thing left is applying some other dataset. I'm extracting this to another issue. #198 |
Well, there's also
And in fact maybe we can have a single-page Get Started, based on the latest improvements (in #188). Should we reopen and discuss that here, or create a new ticket? |
As I stated above, I don't think it's needed. Better to have it well-separated than squeeze everything onto one page IMO. As an option, I can try to use tabs for all 4 sections, and we can TAL how it can look like. But don't like it since the page will be huge. We have this ticket to try to have a single "easy config" page #190 - that sounds similar to what you want here, but it's a different thing and task. |
Yes but I discussed this with @shcheklein and we thought it would be worth it exploring that route (at some point, probably most interesting along with #198). One way to put it is that "MLEM is not DVC", referring to the fact that we originally tried copying DVC docs structure and length including the Get Started, but really the main value of the tool can probably be delivered (main purpose of initiation-type docs) in a shorter form.
No need to try this IMO -- it just moves the navigation from one place to another. I'm thinking of another GS rewrite, basically. May not be an immediate priority. |
Having hard time imagining how to squeeze everything in the current GS in one page. How do you see this GS then? Showing the save+deployment part only? Or what? |
Not sure, depends what's MLEM's core value proposition. Probably just focus on save (bootstrap) and deploy, and mention the other workflows (apply/serve/build) in passing, but move the content to longer guides. But anyway, for now I'd recommend that we try to get the new GS merged: #188 (comment) seems a bit stuck. |
This is Quick Start to me. It's what we have in README rn. Yep, #188 will be merged soon. @madhur-tandon is working on it. |
Getting back to this issue with feedback from @omesser. If I got @omesser correctly, GS feels "disconnected". You open https://mlem.ai/doc/get-started, save the model, and then you can close it without using MLEM to apply/build/deploy models. So https://mlem.ai/doc/get-started page should have all those examples (?) or maybe some of them. TBH, this looks like Quick Start for me, not like Get Started. In MLEM repo, we have this brief quick start https://github.com/iterative/mlem#usage. WDYT? @jorgeorpinel @omesser |
This idea seems similar to the one check box still open above:
If a "quick start" (one-pager) is enough, then why not? But if not, 2 is also better than 4. *Definition-wise, to me Quick Start only hooks you but it's not complete: you need to go elsewhere to finish the story. |
General points to address (may require partial or full rewrite):
mlem init
(copied from DVC) is not needed. GS is should showcase the happy path.i. Capturing models (saving + packaging?)
ii. Productionizing (applying, serving, deploying)
if __name__ == "__main__":
and other code for brevity.Code may be not reproducible if it's shorter: We may show full examples further down (scroll-to) or collapsible sections. Or nowhere (there's an example repo for this)
Extracted to Use more realistic dataset in docs #198iris
dataset looks very toyish. Better to swap it with a more realistic dataset.git commit
commands). Link to the tags where needed instead.The text was updated successfully, but these errors were encountered: