Skip to content

Conversation

rlazo
Copy link
Collaborator

@rlazo rlazo commented Nov 4, 2024

If an invalid location is used when creating the FirebaseVertexAI
object, all requests will fail. The error returned is 404 and HTML
content. The message as-is is not easy to read.

The new messaging points to the most likely reason for the
404 (invalid location).

rlazo added 2 commits November 4, 2024 18:46
If an invalid location is used when creating the `FirebaseVertexAI`
object, all requests will fail. The error returned is 404 and HTML
content. The message as-is is not easy to read.

The new messaging points to the most likely reason for the
404 (invalid location).
Copy link
Contributor

github-actions bot commented Nov 4, 2024

Release note changes

The following release notes were modified. Please ensure they look correct.

Release Notes
firebase-vertexai
### {{firebase_vertexai}} version 16.0.2 {: #vertex-ai_v16-0-2}

* {{fixed}} Improved error message when using an invalid location. GitHub [#6428](//github.com/firebase/firebase-android-sdk/issues/6428){: .external}

* {{fixed}} Fixed issue where Firebase App Check error tokens were unintentionally missing from the requests. GitHub [#6409](//github.com/firebase/firebase-android-sdk/issues/6409){: .external}

* {{fixed}} Clarified in the documentation that `Schema.integer` and `Schema.float` only provide hints to the model. GitHub [#6420](//github.com/firebase/firebase-android-sdk/issues/6420){: .external}

Copy link
Contributor

github-actions bot commented Nov 4, 2024

Vertex AI Mock Responses Check ⚠️

A newer major version of the mock responses for Vertex AI unit tests is available. update_responses.sh should be updated to clone the latest version of the responses: v5.1

Copy link
Contributor

github-actions bot commented Nov 4, 2024

Test Results

 20 files   -  1 002   20 suites   - 1 002   14s ⏱️ - 35m 48s
113 tests  -  5 696  113 ✅  -  5 674  0 💤  - 22  0 ❌ ±0 
226 runs   - 11 477  226 ✅  - 11 433  0 💤  - 44  0 ❌ ±0 

Results for commit 9a1c9b1. ± Comparison against base commit 442bab2.

This pull request removes 5698 and adds 2 tests. Note that renamed tests count towards both.
com.google.android.datatransport.cct.CctBackendFactoryTest ‑ create_returnCCTBackend_WhenBackendNameIsCCT
com.google.android.datatransport.cct.CctDestinationTest ‑ cctDestination_shouldOnlySupportProtoAndJson
com.google.android.datatransport.cct.CctDestinationTest ‑ cctDestination_shouldSupportProtoAndJson
com.google.android.datatransport.cct.CctTransportBackendTest ‑ decorate_whenOffline_shouldProperlyPopulateNetworkInfo
com.google.android.datatransport.cct.CctTransportBackendTest ‑ decorate_whenOnline_shouldProperlyPopulateNetworkInfo
com.google.android.datatransport.cct.CctTransportBackendTest ‑ schedule_shouldAddCookieOnPseudonymousIds
com.google.android.datatransport.cct.CctTransportBackendTest ‑ schedule_shouldDropCookieOnMixedPseudonymousIds
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_CompressedResponseIsUncompressed
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_whenBackendRedirectsMoreThan5Times_shouldOnlyRedirect4Times
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_whenBackendRedirects_shouldCorrectlyFollowTheRedirectViaPost
…
com.google.firebase.vertexai.GenerativeModelTesting ‑ exception thrown when using invalid location
com.google.firebase.vertexai.GenerativeModelTesting ‑ system calling in request

♻️ This comment has been updated with latest results.

@rlazo
Copy link
Collaborator Author

rlazo commented Nov 5, 2024

cc @andrewheard

@rlazo rlazo merged commit b9013d5 into main Nov 5, 2024
27 checks passed
@rlazo rlazo deleted the rl.invalid.location.error branch November 5, 2024 16:03
@firebase firebase locked and limited conversation to collaborators Dec 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants