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 33 minutes and 11 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 a3b1b56b2b88b084744258d590b5259cd55cfcad and a131919ce9550164dc793cc5cec7e198bf1d5eaa.
The changes primarily involve modifying import statements across various files in the deepgram
module to reference the verboselogs
module from a different, more appropriate location. This involves switching from absolute imports to relative imports, ensuring consistency and potentially improving maintainability and readability of the codebase.
Files/Paths | Change Summary |
---|---|
deepgram/__init__.py |
Changed import from from .client import verboselogs to from .utils.verboselogs import * . |
deepgram/audio/microphone/constants.py , deepgram/audio/microphone/microphone.py |
Updated import from deepgram.utils to relative ...utils . |
deepgram/client.py |
Updated import from deepgram.utils to .utils . |
deepgram/clients/analyze/v1/async_client.py , deepgram/clients/analyze/v1/client.py , deepgram/clients/analyze/v1/options.py |
Updated import from deepgram.utils to ....utils . |
deepgram/clients/listen.py , deepgram/clients/read.py |
Updated import from deepgram.utils to ..utils . |
deepgram/clients/live/v1/async_client.py , deepgram/clients/live/v1/client.py |
Updated import from deepgram.utils to ....utils . |
deepgram/clients/manage/v1/async_client.py , deepgram/clients/manage/v1/client.py |
Updated import from deepgram.utils to ....utils . |
deepgram/clients/prerecorded/v1/async_client.py , deepgram/clients/prerecorded/v1/client.py |
Updated import from deepgram.utils to ....utils . |
deepgram/clients/selfhosted/v1/async_client.py , deepgram/clients/selfhosted/v1/client.py |
Updated import from deepgram.utils to ....utils . |
deepgram/clients/speak/v1/async_client.py , deepgram/clients/speak/v1/client.py , deepgram/clients/speak/v1/options.py |
Updated import from deepgram.utils to ....utils . |
deepgram/options.py |
Updated import from deepgram.utils to .utils . |
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 fixes the import of the verbose logs package to resolve the missing dependency. Verified broken in test/dev build
dev.1
.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
NA
Summary by CodeRabbit
verboselogs
module across multiple files to use relative imports. This change improves the maintainability and readability of the codebase.