-
Notifications
You must be signed in to change notification settings - Fork 777
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
Improvements for Kubeflow Pipelines documentation #3712
Comments
@hbelmiro can you please elaborate on the third bullet? Do you have something specific in mind? |
@StefanoFioravanzo Done. |
@hbelmiro @diegolovison Hey, I'd like to draw your attention to these two comments:
and then the Katib PR that addresses those comments: As you can see, I am proposing a restructuring based on principles that we can apply for KFP as well. Would either one of you take the lead and propose a similar restructuring for the KFP docs? It would be great to start with a proposal similar to what I did for Katib, outlining what needs to change, what needs to move, what new sections we need. As you go through this exercise, you can also take note of what user guides or topics are missing, and share a separate list of TODOs. We can track this here #3716 if you want. Whatever suits you better. Let me know your thoughts! PS: My thought process follows this doc framework https://diataxis.fr/ - I encourage you to take ~1h and go through it. It's very well written and easy to follow. It won't take long! |
@StefanoFioravanzo I really like the idea. It will enhance the documentation's quality and keep consistency between the components. We could also mention that structure in https://www.kubeflow.org/docs/about/style-guide/.
Yes, I can do it.
Yeah, sounds good. |
@StefanoFioravanzo I added as a new bullet since it can be done in a follow-up PR. |
/area pipelines |
We should also update the link in the It should be updated to point to the last V1 version: |
We should place a warning at the top of all pages in the This page is for Kubeflow Pipelines V1.
While the V2 backend is [backwards compatible](xxx_v1_v2_compatibility_xxxx) with V1 pipelines,
we recommend you [upgrade to the V2 SDK](xxxx_v2_migration_guide_xxxx), please see the [V2 docs](xxxx_v2_docs_xxxx) for more information. The |
Issues found during #3737
/pipelines/legacy-v1/overview/multi-user/
->/pipelines/operator-guides/multi-user/
#3749The text was updated successfully, but these errors were encountered: