Skip to content

feat: add a way to easily clone a PipelineJob#1239

Merged
jaycee-li merged 17 commits into
googleapis:mainfrom
jaycee-li:main
Jun 3, 2022
Merged

feat: add a way to easily clone a PipelineJob#1239
jaycee-li merged 17 commits into
googleapis:mainfrom
jaycee-li:main

Conversation

@jaycee-li
Copy link
Copy Markdown
Contributor

@jaycee-li jaycee-li requested a review from sasha-gitg May 17, 2022 22:07
@jaycee-li jaycee-li self-assigned this May 17, 2022
@product-auto-label product-auto-label Bot added size: l Pull request size is large. api: vertex-ai Issues related to the googleapis/python-aiplatform API. labels May 17, 2022
Comment thread google/cloud/aiplatform/pipeline_jobs.py Outdated
Comment thread google/cloud/aiplatform/pipeline_jobs.py
Copy link
Copy Markdown
Contributor

@connor-mccarthy connor-mccarthy left a comment

Choose a reason for hiding this comment

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

This looks great. I only have a few small nits and suggestions.

Comment thread google/cloud/aiplatform/pipeline_jobs.py Outdated
Comment thread google/cloud/aiplatform/pipeline_jobs.py Outdated
Comment thread google/cloud/aiplatform/pipeline_jobs.py Outdated
Comment thread google/cloud/aiplatform/pipeline_jobs.py
Comment thread google/cloud/aiplatform/pipeline_jobs.py Outdated
Comment thread google/cloud/aiplatform/pipeline_jobs.py
@jaycee-li jaycee-li merged commit efaf6ed into googleapis:main Jun 3, 2022
rosiezou pushed a commit that referenced this pull request Jun 16, 2022
* Add batch_size kwarg for batch prediction jobs

* Fix errors

Update the copyright year. Change the order of the argument. Fix the syntax error.

* fix: change description layout

* feat: add clone method to PipelineJob

* fix: blacken and lint

* Update pipeline_jobs.py

* fix: update library names

* fix: formatting error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: vertex-ai Issues related to the googleapis/python-aiplatform API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants