fix: Fix Python SDK installation command

[terrytangyuan]

The package is not in PyPI so pip install kubeflow-training won't work and the best way to install is from Git with subdirectory=sdk/python.

Signed-off-by: Yuan Tang [email protected]

[google-oss-robot]

[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:

• OWNERS [terrytangyuan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Jeffwan]

I think the plan is to have this package avialable in pypi but I am not the original process. Jinchi owned this part.

[terrytangyuan]

Hmm this is in the README though so users would expect this to just work. We should only install from PyPI directly when it’s available.

[alembiewski]

@Jeffwan, @terrytangyuan I published the package to testing PyPI for everyone to try it out:
https://test.pypi.org/project/kubeflow-training/
It can be installed and tested from there. When we are ready, let's publish it to the PyPI. I would propose to prioritize publishing it, so the doc update is not needed.
Please let me know in case I can help with releasing it to PyPI.

[terrytangyuan]

We can provide instructions on both so as soon as the package is on PyPI we can add that instruction.

[terrytangyuan]

We may need additional time to test it out to avoid issues like #1439 so it’s better to provide instruction that’s working now.

[Jeffwan]

I thought sdk test case should catch that issue. not sure why it didn't. anyway, I can spend some time on the testing this weekend. After the verification, we can publish it officially and we don't need this PR?

[terrytangyuan]

Sounds good

[terrytangyuan]

Closing as no longer needed