Skip to content
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

Update README.md (mlem branch) #15

Closed
wants to merge 6 commits into from
Closed

Update README.md (mlem branch) #15

wants to merge 6 commits into from

Conversation

jorgeorpinel
Copy link

@jorgeorpinel jorgeorpinel commented Nov 23, 2022

Comment on lines +42 to +44
## To see MLEM in action

...
Copy link
Author

@jorgeorpinel jorgeorpinel Nov 23, 2022

Choose a reason for hiding this comment

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

Write the MLEM section

This

Copy link
Author

Choose a reason for hiding this comment

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

@aguschin do you think we should include some extra info to the README in this branch about MLEM? Maybe mlem pprint something or/and mlem apply/build/serve one of the models?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, we should. Since we create a MLEM deployment, I think we should give a link to a deployed service. Maybe something else, not sure.

Copy link
Contributor

Choose a reason for hiding this comment

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

Added a link https://github.com/iterative/example-repos-dev/pull/149/files

Once we'll have a GTO+MLEM integration page, we can add a link. And maybe something like mlem apply-remote, dunno yet.

@jorgeorpinel jorgeorpinel mentioned this pull request Nov 23, 2022
1 task
@jorgeorpinel jorgeorpinel changed the title Update README.md Update README.md (mlem) Nov 23, 2022
@jorgeorpinel jorgeorpinel changed the title Update README.md (mlem) Update README.md (mlem branch) Nov 23, 2022
@aguschin aguschin closed this Nov 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants