vertex-ai-samples icon indicating copy to clipboard operation
vertex-ai-samples copied to clipboard

Vertex AI Experiments - Model training - Notebook review

Open inardini opened this issue 2 years ago • 1 comments

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

  • [x] Use the notebook template as a starting point.
  • [x] Follow the style and grammar rules outlined in the above notebook template.
  • [x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
  • [x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
  • [x] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
  • [x] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
  • [x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

inardini avatar Aug 11 '22 08:08 inardini

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Ready to be reviewed. Notice that the PR consider two files even if build_model_experimentation_lineage_with_prebuild_code.ipynb is the one to update. For some reason the linter code consider both of files when I launch it. Any ideas?

inardini avatar Aug 14 '22 13:08 inardini

Ready to be reviewed

inardini avatar Aug 17 '22 05:08 inardini