-
Notifications
You must be signed in to change notification settings - Fork 612
Davidmotson.firebase ai #6911
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
Davidmotson.firebase ai #6911
Conversation
…ort (#6907) Co-authored-by: David Motsonashvili <[email protected]>
Vertex AI Mock Responses Check
|
Generated by 🚫 Danger |
Size Report 1Affected Products
Test Logs |
Test Results 1 056 files + 1 038 1 056 suites +1 038 35m 23s ⏱️ + 35m 9s Results for commit bffe50a. ± Comparison against base commit 12127e6. ♻️ This comment has been updated with latest results. |
Coverage Report 1This report is too large (69,887 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.Test Logs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Left some comments, but otherwise LGTM. I defer to @rlazo for final approval tho.
firebase-ai/src/main/kotlin/com/google/firebase/ai/FirebaseAIMultiResourceComponent.kt
Outdated
Show resolved
Hide resolved
firebase-ai/src/main/kotlin/com/google/firebase/ai/common/APIController.kt
Show resolved
Hide resolved
firebase-ai/src/main/kotlin/com/google/firebase/ai/FirebaseAI.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Daymon <[email protected]>
Co-authored-by: Daymon <[email protected]>
…ultiResourceComponent.kt Co-authored-by: Daymon <[email protected]>
Co-authored-by: Daymon <[email protected]>
Citation test still times out, but this address all other breakages.
No description provided.