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

Automl ga base samples #2613

Merged
merged 27 commits into from
Jan 28, 2020
Merged

Automl ga base samples #2613

merged 27 commits into from
Jan 28, 2020

Conversation

nnegrey
Copy link
Contributor

@nnegrey nnegrey commented Dec 13, 2019

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 13, 2019
@nnegrey nnegrey marked this pull request as ready for review January 9, 2020 17:49
@nnegrey nnegrey requested a review from a team as a code owner January 9, 2020 17:49
automl/cloud-client/get_operation_status_test.py Outdated Show resolved Hide resolved
automl/cloud-client/get_operation_status_test.py Outdated Show resolved Hide resolved
automl/cloud-client/get_operation_status_test.py Outdated Show resolved Hide resolved
@nnegrey
Copy link
Contributor Author

nnegrey commented Jan 21, 2020

@kurtisvg, tests outside this PR are running and failing like different products (storage and dialogflow), is this behavior intended?

(I'll look at dialogflow, but mostly wondering about other tests running)

@nnegrey
Copy link
Contributor Author

nnegrey commented Jan 21, 2020

dialogflow was just on the old tests so updating the branch should fix those.

@nnegrey
Copy link
Contributor Author

nnegrey commented Jan 21, 2020

Well ok one test Failed: google.api_core.exceptions.ResourceExhausted: 429 There are too many import data operations are running.

@nnegrey
Copy link
Contributor Author

nnegrey commented Jan 23, 2020

Alright, looks like I've fixed the tests.
@kurtisvg, mind taking a look again?

@nnegrey nnegrey requested a review from kurtisvg January 28, 2020 16:40
automl/cloud-client/get_operation_status_test.py Outdated Show resolved Hide resolved
automl/cloud-client/get_operation_status_test.py Outdated Show resolved Hide resolved
@nnegrey nnegrey merged commit 899cee8 into master Jan 28, 2020
@nnegrey nnegrey deleted the automl-ga-base-samples branch January 28, 2020 20:52
busunkim96 pushed a commit to busunkim96/python-automl that referenced this pull request Aug 7, 2020
…les#2613)

* automl: add base samples

* automl: add base set of samples

* Clean up tests

* License year 2020, drop python2 print statement unicode

* use centralized automl testing project

* Fix GCS path typo

* Use fake dataset for batch predict

* lint: line length

* fix fixture naming and use

* Fix fixture changes

* Catch resource exhausted error

* use fake data for import test

* update how to access an operation id

Co-authored-by: Leah E. Cole <6719667+leahecole@users.noreply.github.com>
busunkim96 pushed a commit to googleapis/python-automl that referenced this pull request Aug 11, 2020
…les#2613)

* automl: add base samples

* automl: add base set of samples

* Clean up tests

* License year 2020, drop python2 print statement unicode

* use centralized automl testing project

* Fix GCS path typo

* Use fake dataset for batch predict

* lint: line length

* fix fixture naming and use

* Fix fixture changes

* Catch resource exhausted error

* use fake data for import test

* update how to access an operation id

Co-authored-by: Leah E. Cole <6719667+leahecole@users.noreply.github.com>
arbrown pushed a commit that referenced this pull request Nov 17, 2022
* automl: add base samples

* automl: add base set of samples

* Clean up tests

* License year 2020, drop python2 print statement unicode

* use centralized automl testing project

* Fix GCS path typo

* Use fake dataset for batch predict

* lint: line length

* fix fixture naming and use

* Fix fixture changes

* Catch resource exhausted error

* use fake data for import test

* update how to access an operation id

Co-authored-by: Leah E. Cole <6719667+leahecole@users.noreply.github.com>
dandhlee pushed a commit that referenced this pull request Nov 17, 2022
* automl: add base samples

* automl: add base set of samples

* Clean up tests

* License year 2020, drop python2 print statement unicode

* use centralized automl testing project

* Fix GCS path typo

* Use fake dataset for batch predict

* lint: line length

* fix fixture naming and use

* Fix fixture changes

* Catch resource exhausted error

* use fake data for import test

* update how to access an operation id

Co-authored-by: Leah E. Cole <6719667+leahecole@users.noreply.github.com>
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Oct 21, 2023
…les#2613)

* automl: add base samples

* automl: add base set of samples

* Clean up tests

* License year 2020, drop python2 print statement unicode

* use centralized automl testing project

* Fix GCS path typo

* Use fake dataset for batch predict

* lint: line length

* fix fixture naming and use

* Fix fixture changes

* Catch resource exhausted error

* use fake data for import test

* update how to access an operation id

Co-authored-by: Leah E. Cole <6719667+leahecole@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants