-
Notifications
You must be signed in to change notification settings - Fork 143
Conversation
7210e9c
to
a83407a
Compare
/cc @gaocegege @johnugeorge |
Thanks for your contribution! 🎉 👍 |
Great. |
@johnugeorge Great comments. Agree with you, |
a83407a
to
e0a5820
Compare
@johnugeorge Updated, Thanks. |
/retest |
@jinchihe Can you please update the PR comment too to reflect the new function names - #240 (comment) |
Updated, thanks! |
Thanks @jinchihe |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: johnugeorge 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 |
/lgtm |
From feedback, as done for the
kubeflow-tfjob
SDK, add more APIs forkubeflow-pytorchjob
:wait_for_job
: Wait for the specified job to finishwait_for_condition
: Waits until any of the specified conditions occurget_job_status
: Quick way to get the PyTorchJob statusis_job_running
: Quick way to check if the PyTorchJob status is Runningis_job_succeeded
: Quick way to check if the PyTorchJob status is Succeeded