-
Notifications
You must be signed in to change notification settings - Fork 775
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
refactor,chore(egen) : refactored code as per template guidelines , performed linter test #3220
refactor,chore(egen) : refactored code as per template guidelines , performed linter test #3220
Conversation
… performed 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.
several needed edits
notebooks/official/ml_metadata/sdk-metric-parameter-tracking-for-custom-jobs.ipynb
Outdated
Show resolved
Hide resolved
notebooks/official/ml_metadata/sdk-metric-parameter-tracking-for-custom-jobs.ipynb
Outdated
Show resolved
Hide resolved
notebooks/official/ml_metadata/sdk-metric-parameter-tracking-for-custom-jobs.ipynb
Outdated
Show resolved
Hide resolved
notebooks/official/ml_metadata/sdk-metric-parameter-tracking-for-custom-jobs.ipynb
Outdated
Show resolved
Hide resolved
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.
small edit
notebooks/official/ml_metadata/sdk-metric-parameter-tracking-for-custom-jobs.ipynb
Outdated
Show resolved
Hide resolved
/gcbrun |
notebooks/official/ml_metadata/sdk-metric-parameter-tracking-for-custom-jobs.ipynb
Show resolved
Hide resolved
/gcbrun |
This PR includes
1.changed REGION to LOCATION
2.removed os module
3.removed os.env(IS_TESTING) in cleanup section
4. performed linter test
5. followed template guidelines
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.