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.

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

[DC-3601] Refactor uses of bq_utils.py get_hpo_bucket_info function #1815

Merged
merged 10 commits into from
Dec 13, 2023

Conversation

lledezma
Copy link
Contributor

No description provided.

@lledezma lledezma changed the title [DC-3601] [DC-3601] Refactor uses of bq_utils.py get_hpo_bucket_info function Nov 30, 2023
@lledezma lledezma marked this pull request as ready for review November 30, 2023 19:33
@ksdkalluri ksdkalluri self-requested a review December 11, 2023 19:22
Copy link
Contributor

@ksdkalluri ksdkalluri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@lledezma lledezma force-pushed the ll/dc-3601 branch 2 times, most recently from 0450a37 to 30e08f9 Compare December 12, 2023 14:46
@lledezma lledezma merged commit a993296 into develop Dec 13, 2023
1 check passed
@lledezma lledezma deleted the ll/dc-3601 branch December 13, 2023 17:45
ratuagga pushed a commit that referenced this pull request Feb 2, 2024
…on (#1815)

* [DC-3601] Add get_hpo_bucket_info method

* [DC-3601] Add HPO_ID and BUCKET_NAME consts

* [DC-3601] Add query template GET_HPO_CONTENTS_QUERY

* [DC-3601] Refactoring

* [DC-3601] Add deprecation warning

* [DC-3601] Add bq_client args

* [DC-3601] Update unit tests

* [DC-3601] Update unit test

* [DC-3601] Cleanup

* [DC-3601] Format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants