Skip to content

chore(genai): replace deprecated genai image generation model#10254

Open
alarconesparza wants to merge 4 commits intoGoogleCloudPlatform:mainfrom
alarconesparza:alarconesparza-update-virtual-try-on-model-505834660
Open

chore(genai): replace deprecated genai image generation model#10254
alarconesparza wants to merge 4 commits intoGoogleCloudPlatform:mainfrom
alarconesparza:alarconesparza-update-virtual-try-on-model-505834660

Conversation

@alarconesparza
Copy link
Copy Markdown
Contributor

@alarconesparza alarconesparza commented Apr 24, 2026

Description

Fixes internal
b/505834660
b/507049205

Checklist

  • I have followed Sample Format Guide
  • pom.xml parent set to latest shared-configuration
  • Appropriate changes to README are included in PR
  • 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)
  • Tests pass: mvn clean verify required
  • Lint passes: mvn -P lint checkstyle:check required
  • Static Analysis: mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
  • 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

@product-auto-label product-auto-label Bot added the samples Issues that are directly related to samples. label Apr 24, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the virtual try-on model ID to 'virtual-try-on-001' in the sample code and integration tests. It also removes the test scope for the 'google-cloud-storage' dependency in the pom.xml. A review comment points out that this dependency should remain in the test scope to avoid unnecessary compile-time dependencies and a larger artifact footprint.

Comment thread genai/snippets/pom.xml
private static final String VIDEO_GEN_MODEL = "veo-3.0-generate-001";
private static final String VIDEO_GEN_PREVIEW_MODEL = "veo-3.0-generate-preview";
private static final String VIDEO_GEN_MODEL = "veo-3.1-generate-001";
private static final String VIDEO_GEN_PREVIEW_MODEL = "veo-3.1-generate-preview";
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It seems that these constant no longer is being used anywhere, since the code line that used it, now points to the general availability model. Since it's not longer needed, line 40 should be removed (or either, line 85, returned to point to the preview model).

@alarconesparza alarconesparza added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 28, 2026
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 28, 2026
@alarconesparza alarconesparza marked this pull request as ready for review April 28, 2026 17:37
@alarconesparza alarconesparza requested review from a team and yoshi-approver as code owners April 28, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants