-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add dsl support for preemptible vm/gpus #1306
Add dsl support for preemptible vm/gpus #1306
Conversation
@Ark-kun @hongye-sun Any concerns about this PR? Thanks |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gaoning777 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 |
/test kubeflow-pipeline-e2e-test |
1 similar comment
/test kubeflow-pipeline-e2e-test |
…e test with sklearn/xgboost examples + docs (kubeflow#1306) * Imported trained model spec and model spec * Update description: trained model -> TrainedModel * Considers model name in predict query * Multi model serving test for sklearn and xgboost * Constants for v1alpha versions * Provider client is generated when downloading * New method to create/deploy trained model object * Example on running multi model serving * Snake case variable -> Camel case * Using new version constants * CreateProviderIfNotExists will return provider and error * Updated to use GetProvider * Corrected the file path * Removed object file path when creating fileName * Added overview of multi-model serving * Overview of inferenceservice, trainedmodel, and model agent * Removed check for version in create_trained_model * Multi-model serving example for sklearn * Moved provider creation to package agent storage * Fixed up confusing wording * Fixed up typo * Included detailed diagram * Added general overview
This change is