AutoML Tables: Enable users to pass in Pandas Dataframe when calling import_data() and batch_predict() from AutoML Tables client#9116
Merged
busunkim96 merged 17 commits intogoogleapis:masterfrom Aug 28, 2019
Conversation
googlebot
added
the
cla: yes
lwander
reviewed
Aug 27, 2019
Contributor
lwander
left a comment
lwander
left a comment
There was a problem hiding this comment.
LG! Just a few small comments
busunkim96
suggested changes
Aug 27, 2019
Contributor
busunkim96
left a comment
busunkim96
left a comment
There was a problem hiding this comment.
👋 I'm not completely sold on the added value of this wrapper. Perhaps this would be better suited for a sample than a library method? @sirtorry?
Contributor
We want to simplify the experience for datascientists in jupyter notebooks as much as possible. Typically they are doing feature engineering & data-processing on dataframes, and we don't want them to have to copy-paste a sample or learn to use another client library for simple things like this. |
lwander
reviewed
Aug 27, 2019
busunkim96
added
the
kokoro:force-run
yoshi-kokoro
removed
the
kokoro:force-run
busunkim96
reviewed
Aug 27, 2019
TrucHLe
commented
Aug 27, 2019
Contributor
Author
TrucHLe
left a comment
TrucHLe
left a comment
There was a problem hiding this comment.
First wave of comments resolved 😃 !
busunkim96
reviewed
Aug 27, 2019
tseaver
added
api: automl
yoshi-kokoro
removed
the
kokoro:force-run
lwander
approved these changes
Aug 28, 2019
Contributor
lwander
left a comment
lwander
left a comment
There was a problem hiding this comment.
LGTM! 2 last comments from my end
busunkim96
added
the
kokoro:force-run
yoshi-kokoro
removed
the
kokoro:force-run
busunkim96
approved these changes
Aug 28, 2019
HemangChothani
pushed a commit
to HemangChothani/google-cloud-python
that referenced
this pull request
Aug 29, 2019
…nd batch_predict() from AutoML Tables client (googleapis#9116)
emar-kar
pushed a commit
to MaxxleLLC/google-cloud-python
that referenced
this pull request
Sep 18, 2019
…nd batch_predict() from AutoML Tables client (googleapis#9116)
parthea
pushed a commit
that referenced
this pull request
Oct 21, 2023
…nd batch_predict() from AutoML Tables client (#9116)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR:
pandas_dataframetoimport_data()andbatch_predict()of the Tables client.