-
Notifications
You must be signed in to change notification settings - Fork 789
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
fix, refactor, chore(egen): removes keras3 dependency error while saving the model, refactors code as per template #3222
Conversation
…ing the model, refactors code as per template, performs linter test
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
/gcbrun |
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.
no edits needed
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
"# Delete Vertex AI pipeline.\n", | ||
"job.delete()\n", | ||
"\n", | ||
"# Delete the CustomJob using its display name.\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.
backticks for CustomJob
??
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.
It is a comment in the code cell. Changed CustomJob to Custom Job.
notebooks/official/tensorboard/tensorboard_vertex_ai_pipelines_integration.ipynb
Outdated
Show resolved
Hide resolved
/gcbrun |
This PR includes the following changes:
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.