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

problems to do batch predictions follow the docs #2500

Closed
alefhsousa opened this issue Oct 28, 2019 · 2 comments
Closed

problems to do batch predictions follow the docs #2500

alefhsousa opened this issue Oct 28, 2019 · 2 comments
Assignees
@sirtorry
Labels
api: automl Issues related to the AutoML API. needs review type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@alefhsousa
Copy link
Contributor

alefhsousa commented Oct 28, 2019 •
edited
Loading

https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/tables/automl/automl_tables_predict.py

Did you change the file? If so, how?

I fixed the problems using the named paremeters. This PR #2501 close the problem

Describe the issue

Today when we try to use the function batch_predict follow the docs we receive and error saying: the paramaters should be a pandas.Dataframe it’s happens because the first parameter of the function batch_predict is a pandas.Dataframe. To solve this problem we need to use de named parameters of python.

image
@yoshi-automation yoshi-automation added triage me I really want to be triaged. 🚨 This issue needs some love. labels Oct 29, 2019
@leahecole leahecole added needs review type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. and removed 🚨 This issue needs some love. triage me I really want to be triaged. labels Nov 15, 2019
@leahecole
Copy link
Collaborator

hi @sirtorry - PTAL - reassign this issue AND the linked PR to a teammate if needed.

@JustinBeckwith JustinBeckwith added the api: automl Issues related to the AutoML API. label Nov 16, 2019
@leahecole
Copy link
Collaborator

leahecole commented Nov 19, 2019 •
edited
Loading

Merged #2501 which fixes this - thanks for the fix!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sirtorry sirtorry

Labels
api: automl Issues related to the AutoML API. needs review type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@JustinBeckwith @leahecole @sirtorry @alefhsousa @yoshi-automation