-
Notifications
You must be signed in to change notification settings - Fork 771
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
chore , refactor : removed ! rm lightweight_pipeline.json , Added cleanup code for deletion of pipeline and refactored code according to template guidelines , performed linter test #3149
chore , refactor : removed ! rm lightweight_pipeline.json , Added cleanup code for deletion of pipeline and refactored code according to template guidelines , performed linter test #3149
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggested edits
notebooks/official/pipelines/lightweight_functions_component_io_kfp.ipynb
Outdated
Show resolved
Hide resolved
notebooks/official/pipelines/lightweight_functions_component_io_kfp.ipynb
Outdated
Show resolved
Hide resolved
"* Vertex AI\n", | ||
"* Cloud Storage\n", | ||
"\n", | ||
"Learn about [Vertex AI\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be put on one line?
"Learn about Vertex AI\n",
"pricing
e.g.,
"Learn about Vertex AI pricing
notebooks/official/pipelines/lightweight_functions_component_io_kfp.ipynb
Outdated
Show resolved
Hide resolved
notebooks/official/pipelines/lightweight_functions_component_io_kfp.ipynb
Outdated
Show resolved
Hide resolved
notebooks/official/pipelines/lightweight_functions_component_io_kfp.ipynb
Outdated
Show resolved
Hide resolved
notebooks/official/pipelines/lightweight_functions_component_io_kfp.ipynb
Outdated
Show resolved
Hide resolved
notebooks/official/pipelines/lightweight_functions_component_io_kfp.ipynb
Outdated
Show resolved
Hide resolved
…ctored code according to template guidelines , performed linter test
…ctored code according to template guidelines , performed linter test
/gcbrun |
…ode according to template guidelines , performed linter test
/gcbrun |
… downgraded numpy version
/gcbrun |
/gcbrun |
Please take a look at the format and lint failures |
/gcbrun |
This PR includes the following updates:
Adds Colab Enterprise link and other structural changes from the new template.
Changes REGION variable name to LOCATION
Adds clean up code for pipeline deletion
Removes IS_TESTING from the cleaning up section.
Removes the usage of future tense.
REQUIRED: Fill out the below checklists or remove if irrelevant
Official Notebooks
under the notebooks/official folder, follow this mandatory checklist:Official Notebooks
section, pointing to the author or the author's team.Community Notebooks
under the notebooks/community folder:Community Notebooks
section, pointing to the author or the author's team.Community Content
under the community-content folder:Content Directory Name
is descriptive, informative, and includes some of the key products and attributes of your content, so that it is differentiable from other contentCommunity Content
section, pointing to the author or the author's team.