-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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(generative-ai): Add compute token examples #12638
Conversation
Add list_tokens_example
Add list_tokens_example
Here is the summary of changes. You are about to add 5 region tags.
You are about to delete 4 region tags.
This comment is generated by snippet-bot.
|
CICD error Need to re-try after sometime. |
It's running 148 tests every time changes get pushed, which might explain the resource exhausted errors. It looks like it's running everything under It would be nice to do the same for all the other subdirectories too. That would avoid running so many redundant tests and would lower resource usage. |
@davidcavazos - Thank you for the detailed steps. Will do as suggested. We are currently working on re-organising the samples into feature folder. go/runnable-snippets-execute
|
Co-authored-by: Holt Skinner <13262395+holtskinner@users.noreply.github.com>
Co-authored-by: Holt Skinner <13262395+holtskinner@users.noreply.github.com>
@msampathkumar thanks for providing the local directory test config. It looks like it's still running everything, not your fault. After closer inspection at the code that detects the test package directory, it looks like it dives deeper from the repo root directory until it finds a
https://github.com/googleapis/btlr/blob/main/cmd/run.go#L325 <- finds most general instead of most specific |
All count token related tests are passing. Error are related to Embedding and FunctionCalling. Re-running the test |
Function calling code sample is failing repeatedly for 002 version and so using 001 |
…_name>` pattern" This reverts commit 663976b.
Raising fresh independent PRs |
Add list_tokens_example
Description
Fixes b358102024
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist
nox -s py-3.9
(see Test Environment Setup)nox -s lint
(see Test Environment Setup)