Skip to content

Conversation

@Valeriy-Burlaka
Copy link
Contributor

@Valeriy-Burlaka Valeriy-Burlaka commented Feb 13, 2025

Description

Adds samples for text generation with the new GenAI SDK and tools. Added region tags:

  • googlegenaisdk_tools_code_exec_with_txt
  • googlegenaisdk_tools_code_exec_with_txt_local_img
  • googlegenaisdk_tools_func_desc_with_txt
  • googlegenaisdk_tools_google_search_with_txt

Checklist

  • I have followed Contributing Guidelines from CONTRIBUTING.MD
  • Tests pass: go test -v ./.. (see Testing)
  • Code formatted: gofmt (see Formatting)
  • Vetting pass: go vet (see Formatting)
  • These samples need a new API enabled in testing projects to pass (let us know which ones)
  • These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
  • This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
  • This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
  • Please merge this PR for me once it is approved

@Valeriy-Burlaka Valeriy-Burlaka self-assigned this Feb 13, 2025
@Valeriy-Burlaka Valeriy-Burlaka requested review from a team as code owners February 13, 2025 12:15
@snippet-bot
Copy link

snippet-bot bot commented Feb 13, 2025

Here is the summary of changes.

You are about to add 4 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@Valeriy-Burlaka Valeriy-Burlaka marked this pull request as draft February 13, 2025 12:15
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Feb 13, 2025
@Valeriy-Burlaka Valeriy-Burlaka force-pushed the vb/genaisdk-textgen-batch7 branch from 72be84f to 60708a0 Compare February 13, 2025 17:11
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a duplicate, it's already created in #5127

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@holtskinner , I know! All of these PRs are based on #5127, which will be merged first once #5124 (Go version bump to 1.23) is out. I will clean up things before any subsequent merges.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@holtskinner , I cleaned up the diff, — please take another look!

@Valeriy-Burlaka Valeriy-Burlaka marked this pull request as ready for review February 14, 2025 16:46
@telpirion telpirion added the api: vertex-ai Issues related to the Vertex AI API. label Feb 18, 2025
@telpirion telpirion marked this pull request as draft March 13, 2025 16:39
@telpirion
Copy link
Collaborator

I think this PR has been abandoned; let's close it. @Valeriy-Burlaka please reopen when you're ready to resume work. Thanks!

@telpirion telpirion closed this Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: vertex-ai Issues related to the Vertex AI API. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants