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

feat!: move to microgen #61

merged 14 commits into from Sep 16, 2020

feat!: move to microgen #61

merged 14 commits into from Sep 16, 2020


Copy link

@busunkim96 busunkim96 commented Aug 27, 2020

This moves the library to the python microgenerator. The main changes for users are explained in the

With the exception of the laggards in #60, all samples are now updated.

The diff is unfortunately large enough that GitHub makes it harder to review. Reviewers, I'd appreciate it if you could focus on the handwritten bits:

  • samples/
  • google/cloud/automl_v1beta1/services/tables/
  • tests/system/gapic/v1beta1/
  • tests/unit/


  • add (note deleted resource helper methods, filter_ -> filter)

@busunkim96 busunkim96 requested review from sirtorry, telpirion and a team as code owners August 27, 2020 16:08
@busunkim96 busunkim96 requested review from engelke and removed request for a team August 27, 2020 16:08
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 27, 2020
@product-auto-label product-auto-label bot added the api: automl Issues related to the googleapis/python-automl API. label Aug 28, 2020
Copy link

@software-dov software-dov left a comment

Choose a reason for hiding this comment

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

LGTM, just curious as to whether the path functions were unusable.

Copy link
Collaborator Author

@helinwang @sirtorry Could you review the changes to the tables client? Thank you!

Copy link

Hi Bu Sun, Could you add some context about what is microgen to the PR description? Judging by the code change related to tables_client, looks like the generated Python proto binding are now different? Thanks. Show resolved Hide resolved
Copy link
Collaborator Author

Will release tomorrow morning. All docs on cloudsite are pointed to this repo now.

@busunkim96 busunkim96 merged commit 009085e into master Sep 16, 2020
7 checks passed
@busunkim96 busunkim96 deleted the microgen branch September 16, 2020 20:36
@release-please release-please bot mentioned this pull request Sep 16, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Sep 16, 2020
🤖 I have created a release \*beep\* \*boop\* 
## [2.0.0]( (2020-09-16)


* move to microgen (#61)

### Features

* move to microgen ([#61]( ([009085e](

### Bug Fixes

* **translate:** fix a broken test [([#4360](]( ([5f7d141](, closes [#4353](
* `update_column_spec` typo in TablesClient docstring ([#18]( ([9feb4cc](, closes [#17](
* update retry configs ([#44]( ([7df9059](

### Documentation

* add cancel operation sample ([abc5070](
* add samples from tables/automl ([#54]( ([d225a5f](, closes [#2090]( [#2100]( [#2102]( [#2103]( [#2101]( [#2110]( [#2115]( [#2150]( [#2145]( [#2203]( [#2340]( [#2337]( [#2336]( [#2339]( [#2338]( [#2276]( [#2257]( [#2424]( [#2407]( [#2501]( [#2459]( [#2601]( [#2523]( [#2005]( [#3033]( [#2806]( [#3750]( [#3571]( [#3929]( [#4022]( [#4127](

This PR was generated with [Release Please](
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
api: automl Issues related to the googleapis/python-automl API. cla: yes This human has signed the Contributor License Agreement.
None yet

Successfully merging this pull request may close these issues.

None yet

4 participants