-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
feat: Adding API v2 Bigquery export samples #9291
feat: Adding API v2 Bigquery export samples #9291
Conversation
90af478
to
8444a31
Compare
@rafaelMurata @RajviM I see this PR refactors the existing v1 samples. Acc to previous discussions, IIUC, we are trying to create new samples in addition to the existing ones right? |
@rafaelMurata Kindly address the test failures. |
52784f8
to
e1cf1fb
Compare
Done. |
/gcbrun |
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.
Great work on the snippets. Kindly merge once all tests pass.
security-command-center/snippets/src/main/java/vtwo/bigquery/CreateBigQueryExport.java
Outdated
Show resolved
Hide resolved
security-command-center/snippets/src/main/java/vtwo/bigquery/UpdateBigQueryExport.java
Outdated
Show resolved
Hide resolved
…reateBigQueryExport.java Co-authored-by: Sita Lakshmi Sangameswaran <sitalakshmi@google.com>
…pdateBigQueryExport.java Co-authored-by: Sita Lakshmi Sangameswaran <sitalakshmi@google.com>
* feat: refactor text-only Gemini prompt * fix line length errors * update testcase --------- Co-authored-by: Sita Lakshmi Sangameswaran <sitalakshmi@google.com>
…oogleCloudPlatform#9293) * Service accounts snippets rewrote using a new library * PR fix * Added GetServiceAccount and GetServiceAccountKey snippets * PR fix * test fix * PR fix * Moved service accounts snippets to iam/snippets * Removed unused dependency * Updated copyright year * PR fix * Refactored disable/enable keys snippets * PR fix * Test fix * AccessTests fix * Imports fix * stage1_testServiceAccountsList refactoring * Revert "stage1_testServiceAccountsList refactoring" This reverts commit 336fa98. * licence headers update * fix
* Remove the preview package Add a function calling example * Missing license header * chore(Checkstyle tweak) * Add missing region tag Fix typo in project ID string * Use the latest libraries-bom that references the 0.4.0 version of the google-cloud-vertexai SDK Use the same example for the token count as in Python and Golang * feat(vertexai): Multimodal snippets using Gemini 1.5 * feat(vertexai): Multimodal snippets using Gemini 1.5 (checkstyle and header tweaks) * feat(vertexai): Multimodal snippets using Gemini 1.5 (incorporating feedback, fixing region tags, and clarifying comments) * chore(vertexai): Update to the latest version of the libraries-bom * chore(vertexai): tweak tests * feat(vertexai): System instruction example * feat(vertexai): Add region tag for system instruction test * feat(vertexai): Add region tag for system instruction test (comment nit) * feat(vertexai): Add region tag for system instruction test (non-predictable output, removing an assert)
…udPlatform#9318) adds creation of instance and only then creation of database, added after block to close delete instance, db and close all connections
…oogleCloudPlatform#9319) * moved all files from "update IAM policy samples to use GAPIC library" PR to snippets module * merge with main, create TestPermissionsTest * remove unused test case * added license header * fix lint * fix lint
…oogleCloudPlatform#9340) This App.java is referenced in https://cloud.google.com/run/docs/samples/cloudrun-manual-logging#cloudrun_manual_logging-java. But the document does not explain how to instantiate and configure the logger variable. https://cloud.google.com/run/docs/logging#run_manual_logging-java has the answer. So let's add the explanation in the source code comment.
* add sources samples * feat: sources samples v2 * improve readability * improve comments * adjustments for pull request
* add region tag for QuestionAnswer.java * fix: remove extraneous region tags; update comments * fix: ensure that region tag includes the last } --------- Co-authored-by: Jennifer Davis <sigje@google.com>
) * feat(vertexai): Multimodal snippets using Gemini 1.5 * feat(vertexai): Multimodal snippets using Gemini 1.5 (checkstyle and header tweaks) * feat(vertexai): Multimodal snippets using Gemini 1.5 (incorporating feedback, fixing region tags, and clarifying comments) * feat(vertexai): Use gemini-1.5-flash-001 instead of the preview model * Reset the initial backoff time to its original setting
Description
b/335222583
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist
pom.xml
parent set to latestshared-configuration
mvn clean verify
requiredmvn -P lint checkstyle:check
requiredmvn -P lint clean compile pmd:cpd-check spotbugs:check
advisory only