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

New texts and slides on the main page #51

Merged
merged 12 commits into from
May 18, 2022
Merged

New texts and slides on the main page #51

merged 12 commits into from
May 18, 2022

Conversation

aguschin
Copy link
Contributor

No description provided.

@gatsby-cloud
Copy link

gatsby-cloud bot commented May 10, 2022

Gatsby Cloud Build Report

mlem.ai

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 1m

Performance

Lighthouse report

Metric Score
Performance 💚 95
Accessibility 🔶 88
Best Practices 💚 100
SEO 🔶 82

🔗 View full report

@aguschin aguschin mentioned this pull request May 13, 2022
1 task
gatsby-config.js Outdated Show resolved Hide resolved
Copy link
Contributor

@jorgeorpinel jorgeorpinel left a comment

Choose a reason for hiding this comment

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

💅🏼 Copy edits on features

One note though: these are features right? Not exactly phrased as reasons (to answer "why MLEM"). Not sure if that's a problem.

src/components/Home/Features/index.tsx Outdated Show resolved Hide resolved
src/components/Home/Features/index.tsx Outdated Show resolved Hide resolved
src/components/Home/Features/index.tsx Outdated Show resolved Hide resolved
src/components/Home/Features/index.tsx Outdated Show resolved Hide resolved
Copy link
Contributor

@jorgeorpinel jorgeorpinel left a comment

Choose a reason for hiding this comment

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

OK these are the most important texts in the whole site probably so let's give them some extra thought 🙂 Cc @omesser

Comment on lines +22 to +23
bold: 'Save your model with a single command',
text: 'Stick to your training workflow'
Copy link
Contributor

Choose a reason for hiding this comment

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

What command? The gif shows a Python call (model.api.save()), Maybe

Suggested change
bold: 'Save your model with a single command',
text: 'Stick to your training workflow'
bold: 'Save ML models from your Python code.',
text: 'Sticking to your existing training workflow'

Comment on lines +26 to +27
bold: 'Use simple YAML file to save model metadata',
text: 'Use the same human-readable format for any ML framework'
Copy link
Contributor

Choose a reason for hiding this comment

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

I understand these top texts try to show the basic workflow of the tool, right? So maybe

Suggested change
bold: 'Use simple YAML file to save model metadata',
text: 'Use the same human-readable format for any ML framework'
bold: 'Model metadata into YAML automatically.',
text: 'Use the same human-readable format for any ML framework.'

Comment on lines +30 to +31
bold: 'Deploy your model anywhere you want',
text: 'Switch between deployment providers with a single command'
Copy link
Contributor

Choose a reason for hiding this comment

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

💅🏼 punctuation (optional in featured "marketing" texts so up to you, but preferable IMO).

Suggested change
bold: 'Deploy your model anywhere you want',
text: 'Switch between deployment providers with a single command'
bold: 'Deploy your model anywhere you want!',
text: 'Switch between deployment providers with a single command.'

Not sure about the term "deployment provider" though. Production environments? Cloud providers? Servers? Hosting?

src/components/Home/Header/index.tsx Outdated Show resolved Hide resolved
@aguschin aguschin changed the title WIP: New message option3 New texts and slides on the main page May 17, 2022
aguschin and others added 4 commits May 18, 2022 11:24
Co-authored-by: Jorge Orpinel <jorgeorpinel@users.noreply.github.com>
content/home-slides.js Outdated Show resolved Hide resolved
gatsby-config.js Outdated Show resolved Hide resolved
gatsby-config.js Outdated Show resolved Hide resolved
gatsby-config.js Outdated Show resolved Hide resolved
aguschin and others added 2 commits May 18, 2022 16:11
Co-authored-by: Julie <43496356+julieg18@users.noreply.github.com>
@aguschin
Copy link
Contributor Author

@jorgeorpinel, sorry for not accounting for all your feedback, I'm merging this for the sake of time. I've accepted some, it was useful here.
and @julieg18, thank you a lot for your help and quick iterations 🙏

@aguschin aguschin merged commit ef8e5d2 into main May 18, 2022
@aguschin aguschin deleted the new-message-option3 branch May 18, 2022 15:49
@jorgeorpinel
Copy link
Contributor

No worries. Should I make an issue to discuss #51 (review) later? Pretty important texts

@aguschin
Copy link
Contributor Author

Yes, please do @jorgeorpinel 🙏

@jorgeorpinel jorgeorpinel mentioned this pull request May 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants