Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: added support for conditional parameters in hyperparameter tuning #1544

Merged
merged 21 commits into from
Aug 2, 2022
Merged

feat: added support for conditional parameters in hyperparameter tuning #1544

merged 21 commits into from
Aug 2, 2022

Conversation

rosiezou
Copy link
Contributor

@rosiezou rosiezou commented Jul 27, 2022 •
edited

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@rosiezou rosiezou requested a review from a team as a code owner July 27, 2022 21:32
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: vertex-ai Issues related to the googleapis/python-aiplatform API. labels Jul 27, 2022
@rosiezou rosiezou linked an issue Jul 27, 2022 that may be closed by this pull request
Support conditional parameters in hyperparameters_tunning jobs #1189
Closed
@nayaknishant nayaknishant added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 27, 2022
@jaycee-li jaycee-li removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 27, 2022
rosiezou and others added 8 commits July 28, 2022 10:54
@rosiezou
Merge branch 'main' into conditional-hyper-param
f0c7799
@rosiezou
fixing unit tests
1c6ad68
@rosiezou
Merge branch 'main' into conditional-hyper-param
f8b7697
@gcf-owl-bot
🦉 Updates from OwlBot post-processor
c9f2ea8 
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
@gcf-owl-bot
Merge branch 'conditional-hyper-param' of https://github.com/googleap…
193f106 
…is/python-aiplatform into conditional-hyper-param
@gcf-owl-bot
🦉 Updates from OwlBot post-processor
727c6a0 
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
@gcf-owl-bot
Merge branch 'conditional-hyper-param' of https://github.com/googleap…
f3daefe 
…is/python-aiplatform into conditional-hyper-param
@rosiezou
fixed all failing tests
1f14caf
@rosiezou rosiezou requested a review from jaycee-li August 1, 2022 18:38
@rosiezou rosiezou self-assigned this Aug 1, 2022
@rosiezou rosiezou merged commit 744cc38 into main Aug 2, 2022
@rosiezou rosiezou deleted the conditional-hyper-param branch August 2, 2022 07:19
@release-please release-please bot mentioned this pull request Aug 2, 2022
Merged
@release-please release-please bot mentioned this pull request Jun 8, 2023
Closed
@release-please release-please bot mentioned this pull request Aug 25, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaycee-li jaycee-li jaycee-li approved these changes

@dizcology dizcology Awaiting requested review from dizcology

@nayaknishant nayaknishant Awaiting requested review from nayaknishant

Assignees

@rosiezou rosiezou

@jaycee-li jaycee-li

Labels
api: vertex-ai Issues related to the googleapis/python-aiplatform API. size: m Pull request size is medium.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support conditional parameters in hyperparameters_tunning jobs
3 participants
@rosiezou @jaycee-li @nayaknishant