Closed dvonthenen closed 1 week ago
[!WARNING]
Rate limit exceeded
@dvonthenen has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 23 minutes and 41 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 33721b46d2d9865226d2e38789eb9cf312346d63 and bee1f98a997005ff2d7c79e575f2cc6ffe65fc0f.
The updates enhance the Deepgram live client by adding auto-flush functionality and improving event handling and keep-alive mechanisms. Key changes include new methods for message inspection and flushing, updated thread management, and new configuration options. Additionally, new test scripts demonstrate the usage of these features with real-time transcription examples.
File(s) | Change Summary |
---|---|
deepgram/clients/live/v1/async_client.py |
Added auto-flush functionality, new constants, and methods for handling messages and keep-alive signals. Updated thread management and logging. |
deepgram/clients/live/v1/client.py |
Enhanced event handling, introduced auto-flush and keep-alive features, added new constants, methods, and updated thread management. |
deepgram/options.py |
Added new attributes and methods for configuration options, including logging, auto-flush, and keep-alive settings. |
tests/edge_cases/auto_flush/async_microphone_mute/README.md |
Provided setup instructions and prerequisites for using the Deepgram API with real-time transcription. |
tests/edge_cases/auto_flush/async_microphone_mute/main.py |
Implemented a script for real-time transcription using a microphone, with event handling for various speech-related events. |
tests/edge_cases/auto_flush/microphone_mute/README.md |
Provided setup instructions and prerequisites for using the Deepgram API with real-time transcription. |
tests/edge_cases/auto_flush/microphone_mute/main.py |
Implemented a script for real-time transcription using a microphone, with event handling for various speech-related events. |
sequenceDiagram
participant User
participant AsyncLiveClient
participant Server
User->>AsyncLiveClient: start()
AsyncLiveClient->>Server: Connect WebSocket
AsyncLiveClient->>AsyncLiveClient: Start _flush_thread
AsyncLiveClient->>AsyncLiveClient: Start _keep_alive_thread
loop Every HALF_SECOND
AsyncLiveClient->>AsyncLiveClient: _flush()
end
loop Every KeepAlive Interval
AsyncLiveClient->>Server: Send KeepAlive message
end
User->>AsyncLiveClient: send(data)
AsyncLiveClient->>Server: Send data
User->>AsyncLiveClient: finish()
AsyncLiveClient->>AsyncLiveClient: Stop _flush_thread
AsyncLiveClient->>AsyncLiveClient: Stop _keep_alive_thread
AsyncLiveClient->>Server: Close WebSocket
In the code where bytes do flow,
A rabbit's touch makes features grow.
With auto-flush and keep-alive,
Transcriptions now feel so alive.
Events handled with care so fine,
Deepgram's service, oh so divine.
🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Proposed changes
This finished off the implementation of the Finalize functionality. Issue: https://github.com/deepgram/deepgram-python-sdk/issues/351
By implementing the Python version of the auto flush found in the Go SDK here: https://github.com/deepgram/deepgram-go-sdk/pull/237
Types of changes
What types of changes does your code introduce to the community Python SDK? Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.Further comments
What is interesting to note is the time difference in processing between the Threaded and Async IO Tasks. Async IO Tasks can be on SEVERAL orders of magnitude SLOWER than the Threaded client. It was originally the reason why I wanted to deprecate the Async Client because, for IO-intensive apps, it flat out just doesn't make any sense.
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Tests