Skip to content
This repository has been archived by the owner on Aug 10, 2024. It is now read-only.

Add "Usage templates" page #246

Closed
wants to merge 2 commits into from
Closed

Add "Usage templates" page #246

wants to merge 2 commits into from

Conversation

aguschin
Copy link
Contributor

@aguschin aguschin commented Dec 2, 2022

https://mlem-ai-configuration-p-k9xn6s.herokuapp.com/doc/config

WIP draft for #190

Wonder, how to fit <table data | cv | nlp> selector here.


@yathomasi, thanks for #238 (comment), that worked) Don't know what was the reason.

@aguschin aguschin self-assigned this Dec 2, 2022
@shcheklein shcheklein temporarily deployed to mlem-ai-configuration-p-k9xn6s December 2, 2022 13:55 Inactive
@github-actions
Copy link

github-actions bot commented Dec 2, 2022

45395a6

Link Check Report

All 1 links passed!

@jorgeorpinel jorgeorpinel marked this pull request as draft December 2, 2022 17:16
@jorgeorpinel jorgeorpinel added the A: docs Area: user documentation (gatsby-theme-iterative) label Dec 2, 2022
Comment on lines +13 to +15
{
"slug": "config",
"label": "Usage templates",
Copy link
Contributor

@jorgeorpinel jorgeorpinel Dec 2, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be in User Guide I think. Let's use a more consistent slug and label too?

@aguschin
Copy link
Contributor Author

aguschin commented Dec 3, 2022

Ah, didn't commit the page itself 😂

@shcheklein shcheklein temporarily deployed to mlem-ai-configuration-p-k9xn6s December 3, 2022 09:44 Inactive
@aguschin
Copy link
Contributor Author

aguschin commented Dec 5, 2022

The summary of a discussion with @omesser and @mike0sv:

  1. This page tries to achieve two things: show use cases and provide code templates. In case of MLEM, this can be hard to achieve.
  2. The tabs doesn't look balanced: some have a lot of context, some do not.
  3. We can put it in different pages and result of adaptation to those locations may be different. In mlem.ai main page this should be catchy, while in docs it should be more detailed.
  4. Considering we have a lot of pages in docs already and we need to get more attention to MLEM, more useful to put it in mlem.ai main page and have a toggle like in CML https://cml.dev, or have a diagram like in https://lets-unify.ai

Considering this, I'm closing the PR in favor of #248

@jorgeorpinel
Copy link
Contributor

I'm closing the PR in favor of #248

Hmm, should we merge with #190?

@aguschin
Copy link
Contributor Author

Sorry, didn't get you. Do you mean to close #190? Or what?

@jorgeorpinel
Copy link
Contributor

Yes, merge the 2 issues. Is this essentially a duplicate of #190? Thanks

@jorgeorpinel
Copy link
Contributor

p.s. Well, now this is closed but some of the info/comments above could be added to #190 if still needed. Up to you, thanks again

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A: docs Area: user documentation (gatsby-theme-iterative)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants