-
Notifications
You must be signed in to change notification settings - Fork 7.4k
[ALF] Add LiveAPI video sample #2724
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
Conversation
- Modified StreamRealtimeVideoScreen to reduce the camera view to half the screen. - Implemented a frame analyzer in CameraView to capture frames once per second. - Added logging for the size of the captured frame's byte array.
|
/gemini review |
Use the newest firebase-bom
firebase-ai/app/src/main/java/com/google/firebase/quickstart/ai/FirebaseAISamples.kt
Show resolved
Hide resolved
Co-authored-by: Rosário P. Fernandes <[email protected]>
rlazo
left a comment
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.
Revert unneeded changes in the manifest
Phlphat746
left a comment
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.
Messege terbo charg
firebase-ai/app/src/main/java/com/google/firebase/quickstart/ai/FirebaseAISamples.kt
Outdated
Show resolved
Hide resolved
gradle/libs.versions.toml
Outdated
| firebaseBom = "34.5.0" | ||
| kotlin = "2.2.21" | ||
| coreKtx = "1.17.0" | ||
| espressoCore = "3.7.0" | ||
| firebaseBom = "34.4.0" |
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.
We already have the Firebase BoM specified in line 6, we don't need it in line 10 again. (this is also causing the CI failure)
gradle/libs.versions.toml
Outdated
| junit = "4.13.2" | ||
| junitVersion = "1.3.0" | ||
| espressoCore = "3.7.0" | ||
| kotlin = "2.2.20" |
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.
Kotlin is already in line 7
…i/FirebaseAISamples.kt Co-authored-by: Rosário P. Fernandes <[email protected]>
Updated versions for firebaseBom and kotlin.
It adds a new sample for video & audio input, using the realtime video API introduced in the SDk @ firebase/firebase-android-sdk#7480