Add samples for natural language api. #425
Merged
Conversation
This file contains 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
googlebot
added
the
cla: yes
jerjou
force-pushed
the
nl
branch
3 times, most recently
from
71822d8
to
4f07e8c
Compare
| @@ -15,6 +15,7 @@ | |||
| import os | |||
|
|
|||
| import pytest | |||
| import requests | |||
|
|
|||
|
|
|||
| class Namespace: | |||
There was a problem hiding this comment.
unrelated to PR but shouldn't Namespace inherit from object?
|
Updated. |
| [Set Up a Service Account](https://cloud.google.com/natural-language/docs/common/auth#set_up_a_service_account), | ||
| and export the following environment variable: | ||
|
|
||
| ``` |
There was a problem hiding this comment.
does gcloud beta auth application-default login not work btw?
There was a problem hiding this comment.
Not sure. See my comment here, though..
| this will print the top 5 actors with negative sentiment: | ||
|
|
||
| ``` | ||
| $ python main.py --inp entity.json |
There was a problem hiding this comment.
first of all, better to put \ at end of each line to make easier copy paste.
secondly I get
TypeError: coercing to Unicode: need string or buffer, NoneType found
| Please follow the [Set Up Your Project](https://cloud.google.com/natural-language/docs/getting-started#set_up_your_project) | ||
| steps in the Quickstart doc to create a project and enable the | ||
| Cloud Natural Language API. Following those steps, make sure that you | ||
| [Set Up a Service Account](https://cloud.google.com/natural-language/docs/common/auth#set_up_a_service_account), |
There was a problem hiding this comment.
This one also requires you to enable Visino API.
|
Ok we can nitpick code later but I'm fine to merge once TypeError: coercing to Unicode: need string or buffer, NoneType found issue is fixed |
|
And note about enabling Vision API |
jerjou
force-pushed
the
nl
branch
3 times, most recently
from
4729fa0
to
04eac7d
Compare
Change-Id: I089b244f1a0a2210fffab6f5747eed9f6150147a
Change-Id: I8ff59d08f2ae8ce4cb3dd0b57b3548db3d5b8add
Change-Id: I1367429c3ceaefd0b0d36d9d7cad6ca29cbfdd2d
Change-Id: Iecc1327db4aa21eb5ee61ce58bba3d142e600734
Change-Id: I33a6c3259dc5a9b0cc1ad081cd57acc452fc6b1a
Change-Id: Ic2b721f775601d2dfb718be26d425e7c2e91eb8a
Change-Id: I2eac2d41e3f8d96c1d049dbbc7b48c96d5fded0a
Change-Id: I026963be93ca3ef776420807e14abdcfcc5fe95f
Change-Id: Iab58378f40c760e88f2b3cc0806985894ce77e76
|
Merging - we'll add the movie_nl sample in a separate PR |
engelke
pushed a commit
that referenced
this pull request
Nov 28, 2022
- [ ] Regenerate this pull request now. feat: Add tags support feat: Add RelatedAsset and deprecate RelatedAssets for relationship GA *The previous representation of the relationship feature is deprecated and unimplemented. The RelatedAsset message represents the new stable format. PiperOrigin-RevId: 449306805 Source-Link: googleapis/googleapis@3d7bd9d Source-Link: https://github.com/googleapis/googleapis-gen/commit/71a93d05d6076271d04b7592f7fad0d3f0c7a040 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNzFhOTNkMDVkNjA3NjI3MWQwNGI3NTkyZjdmYWQwZDNmMGM3YTA0MCJ9
dandhlee
pushed a commit
that referenced
this pull request
Nov 29, 2022
- [ ] Regenerate this pull request now. feat: Add tags support feat: Add RelatedAsset and deprecate RelatedAssets for relationship GA *The previous representation of the relationship feature is deprecated and unimplemented. The RelatedAsset message represents the new stable format. PiperOrigin-RevId: 449306805 Source-Link: googleapis/googleapis@3d7bd9d Source-Link: googleapis/googleapis-gen@71a93d0 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNzFhOTNkMDVkNjA3NjI3MWQwNGI3NTkyZjdmYWQwZDNmMGM3YTA0MCJ9
8 tasks
telpirion
pushed a commit
that referenced
this pull request
Jan 18, 2023
* docs(samples): Added comments * changed comment
8 tasks
dandhlee
pushed a commit
that referenced
this pull request
Feb 6, 2023
* docs(samples): Added comments * changed comment
telpirion
pushed a commit
that referenced
this pull request
Mar 13, 2023
* docs(samples): Added comments * changed comment
dizcology
pushed a commit
that referenced
this pull request
Jun 16, 2023
9 tasks
dizcology
pushed a commit
that referenced
this pull request
Jul 10, 2023
leahecole
pushed a commit
that referenced
this pull request
Jul 11, 2023
dizcology
pushed a commit
that referenced
this pull request
Sep 11, 2023
…1.2 (#425) Co-authored-by: Anthonios Partheniou <partheniou@google.com>
9 tasks
leahecole
pushed a commit
that referenced
this pull request
Sep 15, 2023
…1.2 (#425) Co-authored-by: Anthonios Partheniou <partheniou@google.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.