-
Notifications
You must be signed in to change notification settings - Fork 2.9k
refactor(vertexai): remove decommisioned generativeaionvertexai_gemini_token_count_advanced region tag
#10298
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -16,7 +16,6 @@ | |
|
|
||
| package vertexai.gemini; | ||
|
|
||
| // [START generativeaionvertexai_non_stream_text_basic] | ||
| import com.google.cloud.vertexai.VertexAI; | ||
| import com.google.cloud.vertexai.api.GenerateContentResponse; | ||
| import com.google.cloud.vertexai.generativeai.GenerativeModel; | ||
|
|
@@ -49,5 +48,4 @@ public static String simpleQuestion(String projectId, String location, String mo | |
| return output; | ||
| } | ||
| } | ||
| } | ||
| // [END generativeaionvertexai_non_stream_text_basic] | ||
| } | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -16,7 +16,6 @@ | |||||||
|
|
||||||||
| package vertexai.gemini; | ||||||||
|
|
||||||||
| // [START generativeaionvertexai_gemini_generate_from_text_input] | ||||||||
| import com.google.cloud.vertexai.VertexAI; | ||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It appears that the region tag
Suggested change
|
||||||||
| import com.google.cloud.vertexai.api.GenerateContentResponse; | ||||||||
| import com.google.cloud.vertexai.generativeai.GenerativeModel; | ||||||||
|
|
@@ -52,5 +51,4 @@ public static String textInput( | |||||||
| return output; | ||||||||
| } | ||||||||
| } | ||||||||
| } | ||||||||
| // [END generativeaionvertexai_gemini_generate_from_text_input] | ||||||||
| } | ||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It appears that the region tag
generativeaionvertexai_non_stream_text_basicwas accidentally removed. The PR description only mentions decommissioninggenerativeaionvertexai_gemini_token_count_advanced. Removing this active region tag will break the Java code snippet on the public documentation page 'Generate Text With a Generative Model'. Please restore this tag.