Add Android views to guides#754
Conversation
mattieruth
force-pushed
the
client-guides-ios
branch
from
d61243c to
2782c90
Compare
mattieruth
force-pushed
the
client-guides-android
branch
from
64b4cb9 to
dbf510c
Compare
|
|
||
| ```kotlin | ||
| dependencies { | ||
| implementation("ai.pipecat:small-webrtc-transport:0.3.7") |
There was a problem hiding this comment.
The version number here should be 1.1.0
| enableMic = true | ||
| ) | ||
|
|
||
| client = RTVIClient(SmallWebRTCTransport.Factory(context, BOT_URL), callbacks, options) |
There was a problem hiding this comment.
The code in this file is out of date -- I think it's using v0.3.7 of the Small WebRTC transport whereas we now have v1.1.0, which has a different API.
There was a problem hiding this comment.
hmm. i think it's this way because our reference docs are all still based on 0.3.7 -- https://docs.pipecat.ai/api-reference/client/android/transports/small-webrtc
let me have claude take a stab at updating all
There was a problem hiding this comment.
ok. well. claude found a ton of stuff... mind taking another look?
| | Piece | Role | | ||
| |---|---| | ||
| | `RTVIClient` | Manages the connection and session lifecycle | | ||
| | `SmallWebRTCTransport.Factory` | Creates the WebRTC peer connection to your bot | |
There was a problem hiding this comment.
Out of date (this is talking about v0.3.7)
|
Mostly looks good, with some out-of-date bits for the Small WebRTC transport. |
mattieruth
force-pushed
the
client-guides-android
branch
from
dbf510c to
d7280f2
Compare
| implementation "ai.pipecat:openai-realtime-webrtc-transport:0.3.7" | ||
| ```kotlin | ||
| dependencies { | ||
| implementation("ai.pipecat:openai-realtime-webrtc-transport:1.2.0") |
There was a problem hiding this comment.
This version doesn't exist, the OpenAI realtime transport is still on 0.3.7.
There was a problem hiding this comment.
Or more specifically, it hasn't been published -- I think it was a work in progress when I was pulled onto other tasks.
| implementation "ai.pipecat:small-webrtc-transport:0.3.7" | ||
| ```kotlin | ||
| dependencies { | ||
| implementation("ai.pipecat:small-webrtc-transport:1.2.0") |
There was a problem hiding this comment.
This version has also never existed
There was a problem hiding this comment.
Current latest is 1.1.0: https://central.sonatype.com/search?q=pipecat
|
|
||
| ```kotlin | ||
| dependencies { | ||
| implementation("ai.pipecat:small-webrtc-transport:1.2.0") |
There was a problem hiding this comment.
This will currently fail
|
I've read through this and I think the code is okay, however it references unreleased versions of two of the transports and so wouldn't work if someone use the code/build script updates today. Probably worth holding off on merging this until I've had a chance to revisit the WIP transports and publish the new version. |
3 participants
No description provided.