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

Machine Learning: Better guidance for intro page, and more #63

Merged
merged 4 commits into from
Mar 11, 2024
Merged

Conversation

amotl
Copy link
Member

@amotl amotl commented Mar 8, 2024

About

This patch gives the "Application Domains" page about "Machine Learning" some more love, in form of guidance.

Preview

https://cratedb-guide--63.org.readthedocs.build/domain/ml/

Thoughts

Content elements are meant to be lean and mean, and we can probe different variants to find an optimal layout, or keep them different for the sake of variety. That's the first proposal, also used at other content sections already 1.

image

/cc @andnig, @karynzv, @marijaselakovic, @ckurze, @matkuliak

Footnotes

  1. Different variants of this element can be explored on the Advanced Time Series Analysis, Time Series Video Tutorials, and Telemetry Data Store pages, coming from Evolve "Features" and "Application Domains" sections #53.

@amotl amotl changed the title Machine Learning: Rework intro page, and more Machine Learning: More guidance for intro page, and more Mar 8, 2024
@amotl amotl changed the title Machine Learning: More guidance for intro page, and more Machine Learning: Better guidance for intro page, and more Mar 8, 2024
@amotl amotl added new content New content being added. guidance Matters of layout, shape, and structure. labels Mar 10, 2024
@amotl amotl marked this pull request as ready for review March 10, 2024 18:41
@amotl amotl requested review from hammerhead and seut and removed request for ckurze and marijaselakovic March 11, 2024 10:10
Copy link
Member

@hammerhead hammerhead left a comment

Choose a reason for hiding this comment

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

Very nice overview and collection of resources 👍

@amotl amotl merged commit 68e5a2f into main Mar 11, 2024
3 checks passed
@amotl amotl deleted the amo/ml-index branch March 11, 2024 11:14
@amotl
Copy link
Member Author

amotl commented Mar 11, 2024

Thanks. Merging right away to provide better guidance to readers without delays. This does not mean the PR can't be used to channel feedback for improvements, to be applied retroactively. Please have your voice on it if you can see spots to improve, or about how to improve in general. Thank you in advance.

@ckurze
Copy link
Contributor

ckurze commented Mar 12, 2024

This looks really nice - thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
guidance Matters of layout, shape, and structure. new content New content being added.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants