[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
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

Improve error message for invalid locations #6428

Merged
merged 4 commits into from
Nov 5, 2024
Merged

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).

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants