-
Notifications
You must be signed in to change notification settings - Fork 832
Update Kubeflow Index Page #3755
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 Kubeflow Index Page #3755
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
I left a few small suggestions though. Totally optional.
content/en/_index.html
Outdated
The Kubeflow project is making AI/ML on Kubernetes simple, portable and scalable. | ||
Our goal is not to recreate other services, but to provide a straightforward way to learn, | ||
deploy, and manage best-in-class open-source systems for AI/ML on any cloud infrastructures. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Kubeflow project is making AI/ML on Kubernetes simple, portable and scalable. | |
Our goal is not to recreate other services, but to provide a straightforward way to learn, | |
deploy, and manage best-in-class open-source systems for AI/ML on any cloud infrastructures. | |
The Kubeflow project makes AI/ML on Kubernetes simple, portable and scalable. | |
Our goal is not to recreate other services, but to provide a straightforward way to learn, | |
deploy, and manage best-in-class open-source systems for AI/ML on any cloud infrastructure. |
Our goal is not to recreate other services, but to provide a straightforward way to learn, | ||
deploy, and manage best-in-class open-source systems for AI/ML on any cloud infrastructures. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our goal is not to recreate other services, but to provide a straightforward way to learn, | |
deploy, and manage best-in-class open-source systems for AI/ML on any cloud infrastructures. | |
Our goal is not to recreate other services, but to provide a straightforward way to learn, | |
deploy, and manage best-in-class open-source systems for AI/ML on any cloud infrastructure. |
/lgtm |
@andreyvelich Here are the key points that I think we should hit for the front-page
Here are the key points that I think makes Kubeflow special:
Based on that, here is my proposed wording:
Here is with full HTML:Kubeflow enables machine learning on <a href="https://kubernetes.io/docs/concepts/overview/" target="_blank" rel="noopener">Kubernetes</a> at any scale.
We are an <em>ecosystem</em> of components for each stage in the <a href="/docs/started/architecture/#introducing-the-ml-lifecycle" target="_blank" rel="noopener">machine learning lifecycle</a>.
<br><br>
Anywhere you run Kubernetes, you should be able to <a href="/docs/started/installing-kubeflow/" target="_blank" rel="noopener">install Kubeflow</a>. Here is a screenshot: |
Thank you for proposing this @thesuperzapper, I agree with some statements. Also, we might need to emphasize that Kubeflow is also for artificial intelligence not only for machine learning. For example, Kubeflow components are enabled AI running on the cloud. Any other thoughts from the community @kubeflow/wg-data-leads @kubeflow/wg-pipeline-leads @kubeflow/wg-training-leads @kubeflow/wg-notebooks-leads @kubeflow/wg-manifests-leads @vikas-saxena02 @hbelmiro ? |
proposal iterating from Mathew's suggestion:
I also like the statement about not-recreate components. |
i would like to add to: Here are the key points that I think makes Kubeflow special:
|
I like where @thesuperzapper is going with this. On @andreyvelich 's comments:
You feel like the second sentence does not emphasize this enough?
Do you feel like we should word it as "Kubeflow enables Artificial Intelligence and Machine Learning on Kubernetes ..." ? |
@StefanoFioravanzo Which sentence ?
That shows the value of Kubeflow to develop AI applications. |
The proposed changes looks good to me |
/lgtm |
@vikas-saxena02 which proposal are you supporting? Also, for others, let's continue discussing so we can converge on a wording (rather than LGTM the PR). @andreyvelich perhaps you can mark this PR as a draft to indicate that we are still discussing. |
@thesuperzapper these ones |
As I was saying in #3755 (comment), the core purpose of the intro paragraph is to help users learn about what Kubeflow is:
I am happy for people to suggest changes to the proposal I made. Regarding the specific comments from @andreyvelich:
I think that message is already conveyed by the first paragraph, is there a specific wording change you are suggesting? Also, "Kubernetes" is clearer than the word "Cloud" because Kubernetes can be run anywhere, and cloud is not really defined.
I don't think adding the words "Artificial Intelligence" will add any significant value and may actually detract from the simplicity of the wording. Furthermore, it's a largely undefined term that has lots of emotion associated to it right now. |
I agree with @thesuperzapper and I think the wording he proposed is already a great improvement over what we have now. I share your concerns on the use of "Artificial Intelligence". If the intent of adding "Artificial Intelligence" is to hint at support for GenAI / LLM applications and use cases, I suggest thinking about a new section in the index page, maybe under components, that highlights supported use cases. |
@StefanoFioravanzo @thesuperzapper Why we don't want to add acronyms in our index page ? For the first statement, why you don't like this statement ?
For the second, I like what @thesuperzapper proposed with this modification:
|
I am usually a strong opponent of unnecessary acronyms and especially abbreviations because they lead to very ugly code and hurt the flow of text/speech, but in this case i am supporting the Idea of introducing them once at the beginning of the page and then just using them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am okay with the use of "AI/ML" here for simplicity. They are very widely used terms nowadays. Spelling them out looks a bit mouthful to me.
My vote is just to say "machine learning" as it comprehensively describes the purpose of Kubeflow. I don't think "artificial intelligence" is a usefully defined term at this point (and even less so "AI"), it's really just a buzzword, and using it makes us subject to confusion as the term evolves. My proposal is still to use something like was proposed in #3755 (comment). Are there any specific suggestions people have for the following introduction language:
|
I agree with @thesuperzapper above as ML and AI are kind of synonymous but ML is more suited in Kubeflow context as its more appropriate and followed in literature. AI is more or a less buzzword as @thesuperzapper suggested. |
Why was it introduction of Kubeflow Pipelines ? Kubeflow Pipelines project was started only in mid 2018. Simplicity, Portability, and Scalability doesn't apply only for KFP. It affects other parts of Kubeflow projects. For example, Training Operator is used to train LLMs given the fault tolerance and advanced scheduling capabilities. Also, with SDK it is very simple to create training jobs. Similar things apply for KServe for model serving, and Katib for HP tuning. |
@andreyvelich ok, I agree your one sounds better, let's use your sentence as the beginning with a slight variation. I also think is fine to remove the That would leave us with something like this:
|
Small suggestion:
Maybe we want to be explicit with the install but I would generally assume I have to install it...just a thought. |
83cec50
to
7230f6b
Compare
Signed-off-by: Andrey Velichkevich <[email protected]>
Signed-off-by: Andrey Velichkevich <[email protected]>
7230f6b
to
a2d5e7f
Compare
Ok, I updated it. What do you think about this message:
|
@andreyvelich think it's close, but. I don't like the repetition of "Run", and I think the second sentence is a little cumbersome. How about:
Also, make the link around the full "Deploy Kubeflow" so it's clear what that link is about |
Signed-off-by: Andrey Velichkevich <[email protected]>
@thesuperzapper Ok, maybe we should say |
Signed-off-by: Andrey Velichkevich <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andreyvelich I am happy to LGTM once these small style changes are made, but the overall content is good now.
It makes AI/ML on Kubernetes simple, portable, and scalable. We are an _ecosystem_ of Kubernetes | ||
based components for each stage in the ML Lifecycle with support for best-in-class | ||
open source tools and frameworks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can remove the "We are..." sentence, because it effectively repeats the first sentence.
content/en/_index.html
Outdated
<br><br> | ||
Anywhere you are running Kubernetes, you should be able to run Kubeflow. | ||
Deploy <a href="/docs/started/installing-kubeflow/" target="_blank">Kubeflow</a> anywhere you run Kubernetes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should move the "Deploy" within the link so it's clear what the link is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure
Signed-off-by: Andrey Velichkevich <[email protected]>
68378c7
to
7b7ce63
Compare
@andreyvelich thanks for your work on this, I think we are in a pretty good place, and am happy to merge it /lgtm |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: terrytangyuan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks everyone for your review!
/hold cancel
Thanks for completing this. I appreciate that finding the proper words can be challenging. In the future, we might avoid sentences with multiple prepositional phrases i.e. We are an ecosystem of Kubernetes |
I think those sentences don't actually mean the same thing. Specifically, we are not claiming that Kubeflow's tools are "best in class" but that Kubeflow tools support the best open frameworks (e.g. PyTorch, TensorFlow, Spark, ...). |
That's right. We are explaining that Kubeflow deploys those best-in-class OSS tools.
|
@andreyvelich what we have now is more inclusive of tools which are part of the "ecosystem" but not technically developed by the Kubeflow community. Also, I really think the current "We are an ecosystem..." more correctly explains what Kubeflow is, and helps people quickly understand that we're not just one tool. |
Makes sense. |
Followup on @thesuperzapper comment here: #3728 (comment)
I updated Kubeflow index page in this PR. Please let me know what do you think about this statement ?
For inspiration I checked one of the first Kubeflow videos that @aronchick presented before:
https://youtu.be/R3dVF5wWz-g?t=875
/assign @thesuperzapper @kubeflow/kubeflow-steering-committee @StefanoFioravanzo @hbelmiro @franciscojavierarceo
/hold for review