Skip to content

Conversation

@szaher
Copy link
Member

@szaher szaher commented Jul 17, 2025

What this PR does / why we need it:
Fix Links in pyproject.toml to point to this repo

Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):

Fixes #

Checklist:

  • Docs included if any changes are user facing

Signed-off-by: Saad Zaher <szaher@redhat.com>
Copy link
Member

@Electronic-Waste Electronic-Waste left a comment

Choose a reason for hiding this comment

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

@szaher Thanks for this!
/lgtm
/approve

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Electronic-Waste

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow bot merged commit 74c6b3d into kubeflow:main Jul 17, 2025
3 checks passed
@google-oss-prow google-oss-prow bot added this to the v0.1 milestone Jul 17, 2025
Documentation = "https://www.kubeflow.org/docs/components/trainer/"
Source = "https://github.com/kubeflow/trainer"
Homepage = "https://github.com/kubeflow/sdk"
Documentation = "https://www.kubeflow.org/docs/components"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we plan to add a dedicated section for the SDK in the doc?

@andreyvelich @Electronic-Waste WDYT?

Copy link
Member

@Electronic-Waste Electronic-Waste Jul 17, 2025

Choose a reason for hiding this comment

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

Currently, we planned to add SDK related section in each component (like "how to use BuiltinTrainer" in kubeflow/trainer#2401 (comment)). But I think it should be a great idea to add a dedicated section for SDK in the doc. Since users do not care about the component they use, they only care about how to use it.

Copy link
Member

Choose a reason for hiding this comment

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

Actually, I feel that we should use the project docs for the SDK for now.
Given that SDK is a primary interface for users to interact with Kubeflow projects APIs, why do we want to separate them in the website ? For example, KFP doesn't separate SDK docs from its section.

SDK implements various clients to talk to Kubeflow projects (e.g. TrainerClient, PipelinesClient, OptimizerClient), so users understand with which project they interact with, and know which docs to check.

Additionally, since we say that Python SDK is the primary interface for Kubeflow Trainer, all of the users guides will be using the Kubeflow SDK: https://www.kubeflow.org/docs/components/trainer/user-guides/

We should also talk about it during our Training WG call.

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree that we should have a dedicated SDK section once we're ready for the first SDK release. We'll need a central place to provide an overview of the Kubeflow SDK, what components are included, installation steps and links to each component's user guides. I'm not saying we duplicate content or separate the SDK docs from their components, but we should create one main place where users can start.

We want to promote adoption and if someone asks "how do I get started with the Kubeflow SDK" we don't have a clear page to point them to. Do you want to point them to README from SDK repo instead?

Dedicated SDK section will be a starting point to help users understand the scope and capabilities of the SDK before they dive into specific component documentation.

+1 to discuss this during Training WG call

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants