Skip to content

Prepare VertexAI SDK for release #6941

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

Merged
merged 15 commits into from
May 7, 2025
Merged

Prepare VertexAI SDK for release #6941

merged 15 commits into from
May 7, 2025

Conversation

daymxn
Copy link
Member

@daymxn daymxn commented May 6, 2025

Per b/414829218,

This ensures the Vertex AI SDK is properly aligned for the upcoming release. More specifically, this PR does the following:

@daymxn daymxn requested a review from rlazo May 6, 2025 21:30
@daymxn daymxn self-assigned this May 6, 2025
Copy link
Contributor

github-actions bot commented May 6, 2025

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

@daymxn daymxn enabled auto-merge (squash) May 6, 2025 21:30
@google-oss-bot
Copy link
Contributor

google-oss-bot commented May 6, 2025

Copy link
Contributor

github-actions bot commented May 6, 2025

Test Results

 18 files   -  24   18 suites   - 24   38s ⏱️ -44s
 88 tests  - 236   88 ✅  - 236  0 💤 ±0  0 ❌ ±0 
176 runs   - 484  176 ✅  - 484  0 💤 ±0  0 ❌ ±0 

Results for commit 5518099. ± Comparison against base commit 548dc28.

This pull request removes 324 and adds 88 tests. Note that renamed tests count towards both.
com.google.firebase.remoteconfig.ConfigTests ‑ Custom Signals builder support multiple types
com.google.firebase.remoteconfig.ConfigTests ‑ Firebase#remoteConfig should delegate to FirebaseRemoteConfig#getInstance()
com.google.firebase.remoteconfig.ConfigTests ‑ Firebase#remoteConfig should delegate to FirebaseRemoteConfig#getInstance(FirebaseApp, region)
com.google.firebase.remoteconfig.ConfigTests ‑ FirebaseRemoteConfigSettings builder works
com.google.firebase.remoteconfig.ConfigTests ‑ Overloaded get() operator returns default value when key doesn't exist
com.google.firebase.remoteconfig.ConfigTests ‑ Overloaded get() operator returns value when key exists
com.google.firebase.remoteconfig.CustomSignalsTest ‑ testCustomSignals_builderPutDouble
com.google.firebase.remoteconfig.CustomSignalsTest ‑ testCustomSignals_builderPutDuplicateKeys
com.google.firebase.remoteconfig.CustomSignalsTest ‑ testCustomSignals_builderPutLong
com.google.firebase.remoteconfig.CustomSignalsTest ‑ testCustomSignals_builderPutMixedTypes
…
com.google.firebase.vertexai.GenerativeModelTesting ‑ exception thrown when using invalid location
com.google.firebase.vertexai.GenerativeModelTesting ‑ system calling in request
com.google.firebase.vertexai.SchemaTests ‑ basic schema declaration
com.google.firebase.vertexai.SchemaTests ‑ full schema declaration
com.google.firebase.vertexai.SerializationTests ‑ test Candidate serialization as Json
com.google.firebase.vertexai.SerializationTests ‑ test GenerateContentResponse serialization as Json
com.google.firebase.vertexai.SerializationTests ‑ test Schema serialization as Json
com.google.firebase.vertexai.SerializationTests ‑ test countTokensResponse serialization as Json
com.google.firebase.vertexai.SerializationTests ‑ test modalityTokenCount serialization as Json
com.google.firebase.vertexai.VertexAIStreamingSnapshotTests ‑ citation parsed correctly
…

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented May 6, 2025

Size Report 1

Affected Products

  • firebase-vertexai

    TypeBase (548dc28)Merge (6b8bd18)Diff
    aar800 kB833 kB+33.3 kB (+4.2%)
    apk (aggressive)1.54 MB1.55 MB+7.37 kB (+0.5%)
    apk (release)9.51 MB9.52 MB+10.9 kB (+0.1%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/rIl81g4M0v.html

@daymxn daymxn disabled auto-merge May 7, 2025 15:12
@daymxn daymxn requested a review from rlazo May 7, 2025 19:07
@daymxn daymxn enabled auto-merge (squash) May 7, 2025 20:13
@daymxn daymxn merged commit 7b1855d into main May 7, 2025
30 of 31 checks passed
@daymxn daymxn deleted the daymon-prepare-vertex branch May 7, 2025 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants