-
Notifications
You must be signed in to change notification settings - Fork 114
Update examples latest namespaces #432
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
Update examples latest namespaces #432
Conversation
WalkthroughThe changes primarily involve updating method calls and configurations to adapt to new API endpoints and logging settings. These updates affect the functionality of speech-to-text and text-to-speech processes by altering how connections are established and logs are formatted. Changes
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? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
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.
Actionable comments posted: 3
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (27)
- deepgram/init.py (2 hunks)
- deepgram/client.py (1 hunks)
- deepgram/clients/init.py (5 hunks)
- deepgram/clients/listen/init.py (1 hunks)
- deepgram/clients/listen/client.py (1 hunks)
- examples/advanced/rest/direct_invocation/main.py (2 hunks)
- examples/advanced/websocket/direct_invocation/main.py (2 hunks)
- examples/advanced/websocket/microphone_inheritance/main.py (2 hunks)
- examples/speech-to-text/rest/async_url/main.py (1 hunks)
- examples/speech-to-text/rest/callback/callback/main.py (2 hunks)
- examples/speech-to-text/rest/file/main.py (2 hunks)
- examples/speech-to-text/rest/intent/main.py (2 hunks)
- examples/speech-to-text/rest/legacy_dict_url/main.py (1 hunks)
- examples/speech-to-text/rest/sentiment/main.py (2 hunks)
- examples/speech-to-text/rest/stream_file/main.py (2 hunks)
- examples/speech-to-text/rest/summary/main.py (2 hunks)
- examples/speech-to-text/rest/topic/main.py (2 hunks)
- examples/speech-to-text/rest/url/main.py (1 hunks)
- examples/speech-to-text/websocket/async_http/main.py (1 hunks)
- examples/speech-to-text/websocket/async_microphone/main.py (1 hunks)
- examples/speech-to-text/websocket/http/main.py (1 hunks)
- examples/speech-to-text/websocket/legacy_dict_microphone/main.py (1 hunks)
- examples/speech-to-text/websocket/microphone/main.py (1 hunks)
- examples/text-to-speech/rest/file/async_hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/woodchuck/main.py (1 hunks)
Additional context used
Learnings (7)
examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py (1)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: examples/text-to-speech/rest/file/hello_world_new/main.py:6-6 Timestamp: 2024-07-01T18:17:36.248Z Learning: When reviewing example files in the Deepgram Python SDK, unused imports should not be flagged for removal as they may be needed for quick logging enablement.examples/text-to-speech/rest/file/hello_world/main.py (1)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: examples/text-to-speech/rest/file/hello_world_new/main.py:6-6 Timestamp: 2024-07-01T18:17:36.248Z Learning: When reviewing example files in the Deepgram Python SDK, unused imports should not be flagged for removal as they may be needed for quick logging enablement.deepgram/clients/listen/__init__.py (9)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/websocket/__init__.py:8-8 Timestamp: 2024-07-01T19:21:39.778Z Learning: Unused imports in `deepgram/clients/listen/v1/websocket/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:6-6 Timestamp: 2024-07-01T18:18:02.415Z Learning: Imports for DeepgramClientOptions and ClientOptionsFromEnv in deepgram/clients/listen/__init__.py should not be flagged as unused in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/rest/options.py:12-12 Timestamp: 2024-07-01T19:12:36.972Z Learning: Unused imports in `deepgram/clients/listen/v1/rest/options.py` are retained to maintain backwards compatibility.deepgram/clients/listen/client.py (6)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.deepgram/clients/__init__.py (7)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/__init__.py:16-19 Timestamp: 2024-07-01T19:12:57.715Z Learning: Unused imports in `deepgram/clients/speak/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.deepgram/__init__.py (8)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/websocket/__init__.py:8-8 Timestamp: 2024-07-01T19:21:39.778Z Learning: Unused imports in `deepgram/clients/listen/v1/websocket/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/prerecorded/__init__.py:13-13 Timestamp: 2024-07-01T19:19:26.816Z Learning: Imports for `PreRecordedStreamSource` in `deepgram/clients/prerecorded/__init__.py` should not be flagged as unused in reviews, as they are maintained for backward compatibility.deepgram/client.py (10)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/websocket/__init__.py:8-8 Timestamp: 2024-07-01T19:21:39.778Z Learning: Unused imports in `deepgram/clients/listen/v1/websocket/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#417 File: deepgram/clients/live/v1/client.py:14-14 Timestamp: 2024-06-12T18:02:10.651Z Learning: Ignore suggestions to change import paths to local versions in test cases and examples as per user preference to use the actual `deepgram-sdk` package for testing.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#424 File: deepgram/client.py:81-81 Timestamp: 2024-06-27T00:06:01.811Z Learning: Imports for SpeakStreamClient and AsyncSpeakStreamClient in `deepgram/client.py` are necessary for export purposes and should not be flagged as unused in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/__init__.py:16-19 Timestamp: 2024-07-01T19:12:57.715Z Learning: Unused imports in `deepgram/clients/speak/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.
Ruff
deepgram/clients/listen/__init__.py
54-54:
.client.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
55-55:
.client.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
56-56:
.client.ListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
57-57:
.client.AsyncListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
examples/speech-to-text/rest/stream_file/main.py
37-37: Use context handler for opening files
(SIM115)
examples/advanced/rest/direct_invocation/main.py
8-8:
deepgram.utils.verboselogsimported but unusedRemove unused import:
deepgram.utils.verboselogs(F401)
9-9:
tracebackimported but unusedRemove unused import:
traceback(F401)
deepgram/clients/__init__.py
36-36:
.live.AsyncLiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
40-40:
.listen.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
40-40:
.listen.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
44-44:
.listen.PrerecordedOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
64-64:
.listen.LiveOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
97-97:
.analyze.SyncAnalyzeResponseimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
103-103:
.speak.SpeakOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
109-109:
.speak.SpeakWebSocketEventsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
113-113:
.speak.SpeakClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
114-114:
.speak.SpeakRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
120-120:
.speak.SpeakRESTResponseimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
125-125:
.speak.SpeakWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
126-126:
.speak.AsyncSpeakWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/__init__.py
46-46:
.client.ErrorResponseimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
50-50:
.client.LiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
50-50:
.client.AsyncLiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
51-51:
.client.ListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
51-51:
.client.AsyncListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
52-52:
.client.LiveTranscriptionEventsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
53-53:
.client.LiveOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
66-66:
.client.PreRecordedClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
66-66:
.client.AsyncPreRecordedClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
67-67:
.client.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
67-67:
.client.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
69-69:
.client.PrerecordedOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
70-70:
.client.PreRecordedStreamSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
examples/advanced/websocket/microphone_inheritance/main.py
16-16:
deepgram.LiveResultResponseimported but unusedRemove unused import
(F401)
examples/speech-to-text/websocket/async_http/main.py
44-44: Function definition does not bind loop variable
signal(B023)
examples/speech-to-text/websocket/microphone/main.py
37-37: f-string without any placeholders
Remove extraneous
fprefix(F541)
examples/speech-to-text/websocket/async_microphone/main.py
49-49: f-string without any placeholders
Remove extraneous
fprefix(F541)
deepgram/client.py
36-36:
.clients.LiveClientimported but unusedRemove unused import
(F401)
36-36:
.clients.AsyncLiveClientimported but unusedRemove unused import
(F401)
38-38:
.clients.ListenRESTClientimported but unusedRemove unused import
(F401)
39-39:
.clients.AsyncListenRESTClientimported but unusedRemove unused import
(F401)
40-40:
.clients.ListenWebSocketClientimported but unusedRemove unused import
(F401)
41-41:
.clients.AsyncListenWebSocketClientimported but unusedRemove unused import
(F401)
44-44:
.clients.LiveOptionsimported but unusedRemove unused import
(F401)
45-45:
.clients.LiveTranscriptionEventsimported but unusedRemove unused import
(F401)
Additional comments not posted (33)
examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py (1)
32-32: Update looks good!The change to use
deepgram.speak.rest.v("1")is consistent with the updated client class names.examples/text-to-speech/rest/file/hello_world/main.py (1)
34-34: Update looks good!The change to use
deepgram.speak.rest.v("1")is consistent with the updated client class names.examples/text-to-speech/rest/file/async_hello_world/main.py (1)
32-32: Update looks good!The change to use
deepgram.speak.asyncrest.v("1")is consistent with the updated client class names.deepgram/clients/listen/__init__.py (1)
54-57: New client class imports look good!The new client class imports
ListenRESTClient,AsyncListenRESTClient,ListenWebSocketClient, andAsyncListenWebSocketClientare correctly added. Ensure they are used where necessary to avoid unused import warnings.Tools
Ruff
54-54:
.client.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
55-55:
.client.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
56-56:
.client.ListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
57-57:
.client.AsyncListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
examples/text-to-speech/rest/file/woodchuck/main.py (1)
34-34: LGTM!The method call update aligns with the new client structure.
examples/speech-to-text/rest/async_url/main.py (1)
30-30: LGTM!The method call update aligns with the new client structure.
examples/speech-to-text/rest/legacy_dict_url/main.py (1)
33-33: LGTM!The method call update aligns with the new client structure.
examples/speech-to-text/rest/stream_file/main.py (1)
47-47: LGTM!The method call update aligns with the new client structure.
examples/speech-to-text/rest/url/main.py (1)
28-33: LGTM! But verify the method call.The code changes are approved.
However, ensure that the
transcribe_urlmethod call on therestclass is correct and matches the new client structure.examples/advanced/rest/direct_invocation/main.py (2)
11-11: LGTM!The import statement change to
ListenRESTClientis correct and aligns with the new client structure.
22-28: LGTM! But verify the client instantiation.The code changes are approved.
However, ensure that the
ListenRESTClientinstantiation withClientOptionsFromEnv()is correct and matches the new client structure.examples/speech-to-text/rest/sentiment/main.py (1)
Line range hint
33-50:
LGTM! But verify the method call.The code changes are approved.
However, ensure that the
transcribe_filemethod call on therestclass is correct and matches the new client structure.examples/speech-to-text/rest/topic/main.py (1)
Line range hint
33-50:
LGTM! But verify the method call.The code changes are approved.
However, ensure that the
transcribe_filemethod call on therestclass is correct and matches the new client structure.examples/speech-to-text/rest/intent/main.py (1)
50-50: LGTM! The method call update is correct.The change to use
deepgram.listen.rest.v("1").transcribe_filealigns with the new client structure.examples/speech-to-text/rest/summary/main.py (1)
50-50: LGTM! The method call update is correct.The change to use
deepgram.listen.rest.v("1").transcribe_filealigns with the new client structure.examples/speech-to-text/rest/file/main.py (1)
51-51: LGTM! The method call update is correct.The change to use
deepgram.listen.rest.v("1").transcribe_filealigns with the new client structure.examples/speech-to-text/rest/callback/callback/main.py (2)
54-54: LGTM! The method call update is correct.The change to use
deepgram.listen.rest.v("1").transcribe_file_callbackaligns with the new client structure.
58-58: LGTM! The method call update is correct.The change to use
deepgram.listen.rest.v("1").transcribe_url_callbackaligns with the new client structure.deepgram/clients/listen/client.py (1)
84-85: Ensure backward compatibility.The aliases for
ListenRESTClientandAsyncListenRESTClientare correctly set to the latest versions. This helps maintain backward compatibility.examples/advanced/websocket/direct_invocation/main.py (3)
12-12: Update import statement to useListenWebSocketClient.The import statement has been updated to reflect the new client class name.
26-26: Update comment to reflect new client class.The comment correctly reflects the new client class name
ListenWebSocketClient.
30-34: Update client initialization to useListenWebSocketClient.The client initialization has been updated to use the new
ListenWebSocketClientclass.deepgram/clients/__init__.py (2)
39-41: Update import statements to include new REST client classes.The import statements have been updated to include
ListenRESTClientandAsyncListenRESTClient.Tools
Ruff
40-40:
.listen.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
40-40:
.listen.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
59-61: Update import statements to include new WebSocket client classes.The import statements have been updated to include
ListenWebSocketClientandAsyncListenWebSocketClient.deepgram/__init__.py (2)
49-51: Update import statements to include new WebSocket client classes.The import statements have been updated to include
ListenWebSocketClientandAsyncListenWebSocketClient.Tools
Ruff
50-50:
.client.LiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
50-50:
.client.AsyncLiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
51-51:
.client.ListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
51-51:
.client.AsyncListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
66-67: Update import statements to include new REST client classes.The import statements have been updated to include
ListenRESTClientandAsyncListenRESTClient.Tools
Ruff
66-66:
.client.PreRecordedClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
66-66:
.client.AsyncPreRecordedClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
67-67:
.client.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
67-67:
.client.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
examples/speech-to-text/websocket/legacy_dict_microphone/main.py (1)
32-32: LGTM!The change from
livetowebsocketin the client class name is correct and consistent with the new naming conventions.examples/speech-to-text/websocket/http/main.py (1)
33-33: LGTM!The change from
livetowebsocketin the client class name is correct and consistent with the new naming conventions.examples/advanced/websocket/microphone_inheritance/main.py (1)
13-13: LGTM!The changes from
LiveClienttoListenWebSocketClientand the updates to the constructor are correct and consistent with the new naming conventions.Also applies to: 27-28
examples/speech-to-text/websocket/async_http/main.py (1)
47-47: LGTM!The change from
asynclivetoasyncwebsocketin the client class name is correct and consistent with the new naming conventions.examples/speech-to-text/websocket/microphone/main.py (1)
34-34: Verify the new client usage.Ensure that
deepgram.listen.websocket.v("1")is correctly integrated and tested.examples/speech-to-text/websocket/async_microphone/main.py (1)
46-46: Verify the new client usage.Ensure that
deepgram.listen.asyncwebsocket.v("1")is correctly integrated and tested.deepgram/client.py (1)
36-42: LGTM! Imports are correct.The import statements include the new clients while retaining backward compatibility.
Tools
Ruff
36-36:
.clients.LiveClientimported but unusedRemove unused import
(F401)
36-36:
.clients.AsyncLiveClientimported but unusedRemove unused import
(F401)
38-38:
.clients.ListenRESTClientimported but unusedRemove unused import
(F401)
39-39:
.clients.AsyncListenRESTClientimported but unusedRemove unused import
(F401)
40-40:
.clients.ListenWebSocketClientimported but unusedRemove unused import
(F401)
41-41:
.clients.AsyncListenWebSocketClientimported but unusedRemove unused import
(F401)
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.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (22)
- examples/advanced/rest/direct_invocation/main.py (2 hunks)
- examples/advanced/websocket/direct_invocation/main.py (2 hunks)
- examples/advanced/websocket/microphone_inheritance/main.py (2 hunks)
- examples/speech-to-text/rest/async_url/main.py (1 hunks)
- examples/speech-to-text/rest/callback/callback/main.py (2 hunks)
- examples/speech-to-text/rest/file/main.py (2 hunks)
- examples/speech-to-text/rest/intent/main.py (2 hunks)
- examples/speech-to-text/rest/legacy_dict_url/main.py (1 hunks)
- examples/speech-to-text/rest/sentiment/main.py (2 hunks)
- examples/speech-to-text/rest/stream_file/main.py (1 hunks)
- examples/speech-to-text/rest/summary/main.py (2 hunks)
- examples/speech-to-text/rest/topic/main.py (2 hunks)
- examples/speech-to-text/rest/url/main.py (1 hunks)
- examples/speech-to-text/websocket/async_http/main.py (1 hunks)
- examples/speech-to-text/websocket/async_microphone/main.py (2 hunks)
- examples/speech-to-text/websocket/http/main.py (1 hunks)
- examples/speech-to-text/websocket/legacy_dict_microphone/main.py (1 hunks)
- examples/speech-to-text/websocket/microphone/main.py (2 hunks)
- examples/text-to-speech/rest/file/async_hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/woodchuck/main.py (1 hunks)
Files skipped from review as they are similar to previous changes (19)
- examples/advanced/websocket/direct_invocation/main.py
- examples/speech-to-text/rest/async_url/main.py
- examples/speech-to-text/rest/callback/callback/main.py
- examples/speech-to-text/rest/file/main.py
- examples/speech-to-text/rest/intent/main.py
- examples/speech-to-text/rest/legacy_dict_url/main.py
- examples/speech-to-text/rest/sentiment/main.py
- examples/speech-to-text/rest/stream_file/main.py
- examples/speech-to-text/rest/summary/main.py
- examples/speech-to-text/rest/topic/main.py
- examples/speech-to-text/rest/url/main.py
- examples/speech-to-text/websocket/async_microphone/main.py
- examples/speech-to-text/websocket/http/main.py
- examples/speech-to-text/websocket/legacy_dict_microphone/main.py
- examples/speech-to-text/websocket/microphone/main.py
- examples/text-to-speech/rest/file/async_hello_world/main.py
- examples/text-to-speech/rest/file/hello_world/main.py
- examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py
- examples/text-to-speech/rest/file/woodchuck/main.py
Additional context used
Ruff
examples/advanced/rest/direct_invocation/main.py
8-8:
deepgram.utils.verboselogsimported but unusedRemove unused import:
deepgram.utils.verboselogs(F401)
9-9:
tracebackimported but unusedRemove unused import:
traceback(F401)
examples/advanced/websocket/microphone_inheritance/main.py
16-16:
deepgram.LiveResultResponseimported but unusedRemove unused import
(F401)
examples/speech-to-text/websocket/async_http/main.py
44-44: Function definition does not bind loop variable
signal(B023)
Additional comments not posted (4)
examples/advanced/rest/direct_invocation/main.py (2)
11-11: LGTM! Import change approved.The import statement has been correctly updated to use
ListenRESTClient.
26-28: LGTM! Client instantiation change approved.The client instantiation has been correctly updated to use
ListenRESTClient.examples/advanced/websocket/microphone_inheritance/main.py (1)
27-28: LGTM! Class and constructor change approved.The class definition and constructor have been correctly updated to use
ListenWebSocketClient.examples/speech-to-text/websocket/async_http/main.py (1)
47-47: LGTM! Connection instantiation change approved.The connection instantiation has been correctly updated to use
asyncwebsocket.
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.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (3)
deepgram/clients/__init__.py (1)
61-62: Update the import paths for ListenWebSocketClient and AsyncListenWebSocketClientThe classes
ListenWebSocketClientandAsyncListenWebSocketClientare located indeepgram/clients/listen/v1/websocket/client.pyanddeepgram/clients/listen/v1/websocket/async_client.py, respectively. The import statements indeepgram/clients/__init__.pyshould be updated accordingly.
deepgram/clients/__init__.pyfrom .listen.v1.websocket.client import ListenWebSocketClient from .listen.v1.websocket.async_client import AsyncListenWebSocketClientAnalysis chain
Verify the updated imports for ListenWebSocketClient and AsyncListenWebSocketClient.
Ensure that the imports for
ListenWebSocketClientandAsyncListenWebSocketClientare correct and consistent with the rest of the codebase.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all import statements for `ListenWebSocketClient` and `AsyncListenWebSocketClient` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --type python -A 5 $'from .listen import ListenWebSocketClient, AsyncListenWebSocketClient'Length of output: 126
Script:
#!/bin/bash # Description: Verify all import statements for `ListenWebSocketClient` and `AsyncListenWebSocketClient` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --glob '*.py' -A 5 'from .listen import ListenWebSocketClient, AsyncListenWebSocketClient'Length of output: 416
Script:
#!/bin/bash # Description: Verify the definitions of `ListenWebSocketClient` and `AsyncListenWebSocketClient` in the listen.py file. # Test: Search for the class definitions in the listen.py file. rg --glob 'listen.py' 'class ListenWebSocketClient' rg --glob 'listen.py' 'class AsyncListenWebSocketClient'Length of output: 406
Script:
#!/bin/bash # Description: Verify the definitions of `ListenWebSocketClient` and `AsyncListenWebSocketClient` in the listen.py file. # Locate the `listen.py` file. fd listen.py # If `listen.py` is found, search for the class definitions. if [ -f "deepgram/clients/listen.py" ]; then rg 'class ListenWebSocketClient' deepgram/clients/listen.py rg 'class AsyncListenWebSocketClient' deepgram/clients/listen.py fiLength of output: 54
Script:
#!/bin/bash # Description: Verify the definitions of `ListenWebSocketClient` and `AsyncListenWebSocketClient` in the repository. # Search for the class definitions in the listen.py file. rg 'class ListenWebSocketClient' deepgram/clients/listen.py rg 'class AsyncListenWebSocketClient' deepgram/clients/listen.py # If not found in listen.py, search the entire codebase. if ! rg --quiet 'class ListenWebSocketClient' deepgram/clients/listen.py; then rg 'class ListenWebSocketClient' --glob '*.py' fi if ! rg --quiet 'class AsyncListenWebSocketClient' deepgram/clients/listen.py; then rg 'class AsyncListenWebSocketClient' --glob '*.py' fiLength of output: 903
deepgram/client.py (1)
99-100: Update Usage of Deprecated PropertiesThe following occurrences of deprecated properties
asyncspeak,onprem, andasynconpremneed to be updated to use the new propertiesspeak.asyncrest,speak.selfhosted, andspeak.asyncselfhostedrespectively:
deepgram/client.py
- Line 99:
asyncspeak- Line 104:
onprem- Line 109:
asynconpremexamples/text-to-speech/rest/file/async_hello_world/main.py
- Line 20:
asyncspeakPlease update these instances to ensure the codebase is using the new, recommended properties.
Analysis chain
LGTM! Verify the function usage in the codebase.
The changes to the
DeepgramClientclass are correct and well-implemented. Ensure that all calls to the deprecated propertiesasyncspeak,onprem, andasynconpremare updated to use the new recommended properties.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all function calls to the deprecated properties match the updated usage. # Test: Search for the usage of the deprecated properties. Expect: Only occurrences of the new usage. rg --type python -A 5 $'asyncspeak|onprem|asynconprem'Length of output: 86
Script:
#!/bin/bash # Description: Verify all function calls to the deprecated properties match the updated usage. # Test: Search for the usage of the deprecated properties. Expect: Only occurrences of the new usage. rg -A 5 'asyncspeak|onprem|asynconprem'Length of output: 6907
Tools
Ruff
99-99:
.clients.SpeakRESTOptionsimported but unusedRemove unused import
(F401)
100-100:
.clients.SpeakWebSocketOptionsimported but unusedRemove unused import
(F401)
deepgram/clients/speak/v1/websocket/client.py (1)
Inconsistent
startfunction calls foundThe
startfunction calls across multiple files do not consistently use the updated parameters.
- Files with issues:
tests/expected_failures/exercise_timeout/async/main.pytests/expected_failures/exercise_timeout/sync/main.pytests/edge_cases/reconnect_same_object/async/main.pytests/edge_cases/reconnect_same_object/sync/main.pyexamples/text-to-speech/websocket/interactive/main.pytests/edge_cases/keepalive/sync/main.pyexamples/speech-to-text/websocket/microphone/main.pyexamples/speech-to-text/websocket/async_microphone/main.pyexamples/speech-to-text/websocket/legacy_dict_microphone/main.pyexamples/speech-to-text/websocket/http/main.pyexamples/speech-to-text/websocket/async_http/main.pytests/edge_cases/keepalive/async/main.pyexamples/advanced/websocket/direct_invocation/main.pyexamples/advanced/websocket/mute-microphone/main.pyexamples/text-to-speech/websocket/async_interactive/main.pyexamples/advanced/websocket/microphone_inheritance/main.pyEnsure that all calls to
startin the above files use the correct parameters:options,addons,headers,members, andkwargs.Analysis chain
Line range hint
84-119: LGTM! Verify the function usage in the codebase.The changes to the
startfunction are correct and well-implemented. Ensure that all calls tostartmatch the updated parameters.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all function calls to `start` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg --type python -A 5 $'start'Length of output: 60
Script:
#!/bin/bash # Description: Verify all function calls to `start` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg -t py -A 5 $'start'Length of output: 61128
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (39)
- deepgram/init.py (3 hunks)
- deepgram/client.py (3 hunks)
- deepgram/clients/init.py (4 hunks)
- deepgram/clients/listen/init.py (3 hunks)
- deepgram/clients/listen/client.py (5 hunks)
- deepgram/clients/listen/v1/init.py (2 hunks)
- deepgram/clients/listen/v1/rest/init.py (1 hunks)
- deepgram/clients/listen/v1/rest/options.py (1 hunks)
- deepgram/clients/listen/v1/websocket/init.py (1 hunks)
- deepgram/clients/listen/v1/websocket/options.py (1 hunks)
- deepgram/clients/speak/init.py (1 hunks)
- deepgram/clients/speak/client.py (2 hunks)
- deepgram/clients/speak/v1/init.py (1 hunks)
- deepgram/clients/speak/v1/rest/async_client.py (7 hunks)
- deepgram/clients/speak/v1/rest/client.py (7 hunks)
- deepgram/clients/speak/v1/websocket/async_client.py (4 hunks)
- deepgram/clients/speak/v1/websocket/client.py (4 hunks)
- examples/advanced/rest/direct_invocation/main.py (2 hunks)
- examples/advanced/websocket/direct_invocation/main.py (2 hunks)
- examples/advanced/websocket/microphone_inheritance/main.py (2 hunks)
- examples/speech-to-text/rest/async_url/main.py (1 hunks)
- examples/speech-to-text/rest/callback/callback/main.py (2 hunks)
- examples/speech-to-text/rest/file/main.py (2 hunks)
- examples/speech-to-text/rest/intent/main.py (2 hunks)
- examples/speech-to-text/rest/legacy_dict_url/main.py (1 hunks)
- examples/speech-to-text/rest/sentiment/main.py (2 hunks)
- examples/speech-to-text/rest/stream_file/main.py (1 hunks)
- examples/speech-to-text/rest/summary/main.py (2 hunks)
- examples/speech-to-text/rest/topic/main.py (2 hunks)
- examples/speech-to-text/rest/url/main.py (1 hunks)
- examples/speech-to-text/websocket/async_http/main.py (1 hunks)
- examples/speech-to-text/websocket/async_microphone/main.py (2 hunks)
- examples/speech-to-text/websocket/http/main.py (1 hunks)
- examples/speech-to-text/websocket/legacy_dict_microphone/main.py (1 hunks)
- examples/speech-to-text/websocket/microphone/main.py (2 hunks)
- examples/text-to-speech/rest/file/async_hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/woodchuck/main.py (1 hunks)
Files skipped from review due to trivial changes (3)
- deepgram/clients/listen/v1/rest/options.py
- deepgram/clients/listen/v1/websocket/options.py
- examples/speech-to-text/rest/sentiment/main.py
Files skipped from review as they are similar to previous changes (19)
- deepgram/clients/listen/client.py
- examples/advanced/rest/direct_invocation/main.py
- examples/advanced/websocket/direct_invocation/main.py
- examples/advanced/websocket/microphone_inheritance/main.py
- examples/speech-to-text/rest/async_url/main.py
- examples/speech-to-text/rest/callback/callback/main.py
- examples/speech-to-text/rest/intent/main.py
- examples/speech-to-text/rest/legacy_dict_url/main.py
- examples/speech-to-text/rest/stream_file/main.py
- examples/speech-to-text/rest/summary/main.py
- examples/speech-to-text/rest/topic/main.py
- examples/speech-to-text/rest/url/main.py
- examples/speech-to-text/websocket/async_http/main.py
- examples/speech-to-text/websocket/async_microphone/main.py
- examples/speech-to-text/websocket/http/main.py
- examples/speech-to-text/websocket/legacy_dict_microphone/main.py
- examples/speech-to-text/websocket/microphone/main.py
- examples/text-to-speech/rest/file/hello_world/main.py
- examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py
Additional context used
Learnings (11)
deepgram/clients/listen/v1/websocket/__init__.py (7)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/websocket/__init__.py:8-8 Timestamp: 2024-07-01T19:21:39.778Z Learning: Unused imports in `deepgram/clients/listen/v1/websocket/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/rest/options.py:12-12 Timestamp: 2024-07-01T19:12:36.972Z Learning: Unused imports in `deepgram/clients/listen/v1/rest/options.py` are retained to maintain backwards compatibility.deepgram/clients/listen/v1/rest/__init__.py (8)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/rest/options.py:12-12 Timestamp: 2024-07-01T19:12:36.972Z Learning: Unused imports in `deepgram/clients/listen/v1/rest/options.py` are retained to maintain backwards compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:6-6 Timestamp: 2024-07-01T18:18:02.415Z Learning: Imports for DeepgramClientOptions and ClientOptionsFromEnv in deepgram/clients/listen/__init__.py should not be flagged as unused in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.deepgram/clients/speak/v1/__init__.py (3)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/__init__.py:16-19 Timestamp: 2024-07-01T19:12:57.715Z Learning: Unused imports in `deepgram/clients/speak/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/v1/websocket/__init__.py:11-11 Timestamp: 2024-07-01T19:14:53.172Z Learning: User dvonthenen prefers to retain unused imports in the codebase to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/v1/websocket/__init__.py:14-14 Timestamp: 2024-07-01T19:14:41.918Z Learning: User dvonthenen prefers to retain unused imports in the codebase to maintain backward compatibility.deepgram/clients/listen/__init__.py (8)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/websocket/__init__.py:8-8 Timestamp: 2024-07-01T19:21:39.778Z Learning: Unused imports in `deepgram/clients/listen/v1/websocket/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/rest/options.py:12-12 Timestamp: 2024-07-01T19:12:36.972Z Learning: Unused imports in `deepgram/clients/listen/v1/rest/options.py` are retained to maintain backwards compatibility.deepgram/clients/listen/v1/__init__.py (9)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/rest/options.py:12-12 Timestamp: 2024-07-01T19:12:36.972Z Learning: Unused imports in `deepgram/clients/listen/v1/rest/options.py` are retained to maintain backwards compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/websocket/__init__.py:8-8 Timestamp: 2024-07-01T19:21:39.778Z Learning: Unused imports in `deepgram/clients/listen/v1/websocket/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:6-6 Timestamp: 2024-07-01T18:18:02.415Z Learning: Imports for DeepgramClientOptions and ClientOptionsFromEnv in deepgram/clients/listen/__init__.py should not be flagged as unused in reviews.deepgram/clients/speak/client.py (2)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/__init__.py:16-19 Timestamp: 2024-07-01T19:12:57.715Z Learning: Unused imports in `deepgram/clients/speak/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/v1/websocket/__init__.py:11-11 Timestamp: 2024-07-01T19:14:53.172Z Learning: User dvonthenen prefers to retain unused imports in the codebase to maintain backward compatibility.deepgram/__init__.py (9)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/websocket/__init__.py:8-8 Timestamp: 2024-07-01T19:21:39.778Z Learning: Unused imports in `deepgram/clients/listen/v1/websocket/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:37-38 Timestamp: 2024-07-01T19:13:29.909Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:54-55 Timestamp: 2024-07-01T19:13:11.612Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/__init__.py:36-43 Timestamp: 2024-07-01T19:17:04.194Z Learning: Unused imports in `deepgram/clients/listen/v1/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:42-50 Timestamp: 2024-07-01T19:13:28.504Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:19-24 Timestamp: 2024-07-01T19:14:20.539Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained to maintain backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/__init__.py:10-13 Timestamp: 2024-07-01T19:14:11.334Z Learning: Unused imports in `deepgram/clients/listen/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/prerecorded/__init__.py:13-13 Timestamp: 2024-07-01T19:19:26.816Z Learning: Imports for `PreRecordedStreamSource` in `deepgram/clients/prerecorded/__init__.py` should not be flagged as unused in reviews, as they are maintained for backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/__init__.py:16-19 Timestamp: 2024-07-01T19:12:57.715Z Learning: Unused imports in `deepgram/clients/speak/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.deepgram/clients/speak/v1/rest/client.py (2)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/__init__.py:16-19 Timestamp: 2024-07-01T19:12:57.715Z Learning: Unused imports in `deepgram/clients/speak/__init__.py` are retained for backward compatibility and should not be flagged for removal in reviews.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/listen/v1/rest/options.py:12-12 Timestamp: 2024-07-01T19:12:36.972Z Learning: Unused imports in `deepgram/clients/listen/v1/rest/options.py` are retained to maintain backwards compatibility.deepgram/clients/speak/v1/rest/async_client.py (1)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#424 File: deepgram/client.py:81-81 Timestamp: 2024-06-27T00:06:01.811Z Learning: Imports for SpeakStreamClient and AsyncSpeakStreamClient in `deepgram/client.py` are necessary for export purposes and should not be flagged as unused in reviews.deepgram/clients/speak/v1/websocket/client.py (2)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/v1/websocket/__init__.py:11-11 Timestamp: 2024-07-01T19:14:53.172Z Learning: User dvonthenen prefers to retain unused imports in the codebase to maintain backward compatibility.Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/v1/websocket/__init__.py:14-14 Timestamp: 2024-07-01T19:14:41.918Z Learning: User dvonthenen prefers to retain unused imports in the codebase to maintain backward compatibility.deepgram/clients/speak/v1/websocket/async_client.py (1)
Learnt from: dvonthenen PR: deepgram/deepgram-python-sdk#426 File: deepgram/clients/speak/v1/websocket/__init__.py:11-11 Timestamp: 2024-07-01T19:14:53.172Z Learning: User dvonthenen prefers to retain unused imports in the codebase to maintain backward compatibility.
Ruff
deepgram/clients/listen/v1/websocket/__init__.py
7-7:
.options.LiveOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
7-7:
.options.ListenWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/listen/v1/rest/__init__.py
8-8:
.options.ListenRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/speak/v1/__init__.py
6-6:
.options.SpeakOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
7-7:
.options.SpeakRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
8-8:
.options.SpeakWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
9-9:
.options.FileSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
10-10:
.options.SpeakWebSocketSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
11-11:
.options.SpeakSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/speak/__init__.py
17-17:
.client.SpeakRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
18-18:
.client.SpeakWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/listen/__init__.py
19-19:
.client.ListenRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
38-38:
.client.ListenWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
56-56:
.client.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
57-57:
.client.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/listen/v1/__init__.py
19-19:
.rest.ListenRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
19-19:
.rest.PrerecordedOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
34-34:
.websocket.LiveOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
34-34:
.websocket.ListenWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/__init__.py
40-40:
.listen.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
40-40:
.listen.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
44-44:
.listen.ListenRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
65-65:
.listen.ListenWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
105-105:
.speak.SpeakOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
106-106:
.speak.SpeakRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
107-107:
.speak.SpeakWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
109-109:
.speak.SpeakWebSocketSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
110-110:
.speak.SpeakSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
113-113:
.speak.SpeakWebSocketEventsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/__init__.py
50-50:
.client.LiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
50-50:
.client.AsyncLiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
51-51:
.client.ListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
51-51:
.client.AsyncListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
52-52:
.client.LiveTranscriptionEventsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
53-53:
.client.LiveOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
53-53:
.client.ListenWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
66-66:
.client.PreRecordedClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
66-66:
.client.AsyncPreRecordedClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
67-67:
.client.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
67-67:
.client.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
69-69:
.client.ListenRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
97-97:
.client.SpeakRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
98-98:
.client.SpeakWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/client.py
36-36:
.clients.LiveClientimported but unusedRemove unused import
(F401)
36-36:
.clients.AsyncLiveClientimported but unusedRemove unused import
(F401)
38-38:
.clients.ListenRESTClientimported but unusedRemove unused import
(F401)
39-39:
.clients.AsyncListenRESTClientimported but unusedRemove unused import
(F401)
40-40:
.clients.ListenWebSocketClientimported but unusedRemove unused import
(F401)
41-41:
.clients.AsyncListenWebSocketClientimported but unusedRemove unused import
(F401)
44-44:
.clients.ListenWebSocketOptionsimported but unusedRemove unused import
(F401)
67-67:
.clients.ListenRESTOptionsimported but unusedRemove unused import
(F401)
99-99:
.clients.SpeakRESTOptionsimported but unusedRemove unused import
(F401)
100-100:
.clients.SpeakWebSocketOptionsimported but unusedRemove unused import
(F401)
Additional comments not posted (26)
deepgram/clients/listen/v1/websocket/__init__.py (1)
7-7: Retain unused imports for backward compatibility.Based on previous learnings, unused imports in this file are retained to maintain backward compatibility and should not be flagged for removal.
Tools
Ruff
7-7:
.options.LiveOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
7-7:
.options.ListenWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/listen/v1/rest/__init__.py (1)
8-8: Retain unused imports for backward compatibility.Based on previous learnings, unused imports in this file are retained to maintain backward compatibility and should not be flagged for removal.
Tools
Ruff
8-8:
.options.ListenRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/speak/v1/__init__.py (1)
6-12: Retain unused imports for backward compatibility.Based on previous learnings, unused imports in this file are retained to maintain backward compatibility and should not be flagged for removal.
Tools
Ruff
6-6:
.options.SpeakOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
7-7:
.options.SpeakRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
8-8:
.options.SpeakWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
9-9:
.options.FileSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
10-10:
.options.SpeakWebSocketSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
11-11:
.options.SpeakSourceimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/speak/__init__.py (1)
17-18: Retain unused imports for backward compatibility.Based on previous learnings, unused imports in this file are retained to maintain backward compatibility and should not be flagged for removal.
Tools
Ruff
17-17:
.client.SpeakRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
18-18:
.client.SpeakWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
examples/text-to-speech/rest/file/async_hello_world/main.py (1)
32-32: LGTM!The method call has been correctly updated to
deepgram.speak.asyncrest.v("1").save.examples/text-to-speech/rest/file/woodchuck/main.py (1)
34-34: LGTM!The method call has been correctly updated to
deepgram.speak.rest.v("1").save.deepgram/clients/listen/__init__.py (1)
19-19: Unused imports are retained for backward compatibility.The imports
ListenRESTOptions,ListenWebSocketOptions,ListenRESTClient, andAsyncListenRESTClientare retained intentionally to maintain backward compatibility.Also applies to: 38-38, 56-57
Tools
Ruff
19-19:
.client.ListenRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/listen/v1/__init__.py (1)
19-19: Unused imports are retained for backward compatibility.The imports
ListenRESTOptions,PrerecordedOptions,LiveOptions, andListenWebSocketOptionsare retained intentionally to maintain backward compatibility.Also applies to: 34-34
Tools
Ruff
19-19:
.rest.ListenRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
19-19:
.rest.PrerecordedOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
examples/speech-to-text/rest/file/main.py (1)
51-51: Verify the updated method call.Ensure that the change from
deepgram.listen.live.v("1")todeepgram.listen.rest.v("1")is correct and consistent with the rest of the codebase.deepgram/clients/speak/client.py (1)
13-14: Verify the updated imports.Ensure that the imports from
.v1are correct and consistent with the rest of the codebase.Verification successful
The updated imports are consistent with the rest of the codebase.
- The imports of
SpeakRESTOptionsandSpeakWebSocketOptionsfrom.v1are correctly aliased asSpeakRESTOptionsLatestandSpeakWebSocketOptionsLatestindeepgram/clients/speak/client.py.- These aliases are used appropriately in the codebase.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all import statements from `.v1` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --type python -A 5 $'from .v1 import'Length of output: 72
Script:
#!/bin/bash # Description: Verify all import statements from `.v1` in the codebase. # Find all Python files and search for the import usage in the codebase. fd -e py -x rg -H 'from .v1 import' {}Length of output: 1428
Script:
#!/bin/bash # Description: Verify the usage of SpeakRESTOptions and SpeakWebSocketOptions in the codebase. # Find all occurrences of SpeakRESTOptions and SpeakWebSocketOptions in Python files. fd -e py -x rg -H 'SpeakRESTOptions|SpeakWebSocketOptions' {}Length of output: 4206
deepgram/clients/__init__.py (2)
40-41: Verify the updated imports for ListenRESTClient and AsyncListenRESTClient.Ensure that the imports for
ListenRESTClientandAsyncListenRESTClientare correct and consistent with the rest of the codebase.Verification successful
Verified the updated imports for
ListenRESTClientandAsyncListenRESTClient.The import statements in
deepgram/clients/__init__.pyare correct and consistent with the rest of the codebase.
ListenRESTClientis defined indeepgram/clients/listen/v1/rest/client.py.AsyncListenRESTClientis defined indeepgram/clients/listen/v1/rest/async_client.py.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all import statements for `ListenRESTClient` and `AsyncListenRESTClient` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --type python -A 5 $'from .listen import ListenRESTClient, AsyncListenRESTClient'Length of output: 116
Script:
#!/bin/bash # Description: Verify all import statements for `ListenRESTClient` and `AsyncListenRESTClient` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg -A 5 'from .listen import ListenRESTClient, AsyncListenRESTClient'Length of output: 384
Script:
#!/bin/bash # Description: Verify the definitions of `ListenRESTClient` and `AsyncListenRESTClient` in the listen module. # Test: Search for the class definitions in the listen module. Expect: Definitions of `ListenRESTClient` and `AsyncListenRESTClient`. rg 'class ListenRESTClient' rg 'class AsyncListenRESTClient'Length of output: 256
Script:
#!/bin/bash # Description: Verify if the listen module re-exports `ListenRESTClient` and `AsyncListenRESTClient`. # Test: Search for the re-export statements in the listen module. Expect: Re-exports of `ListenRESTClient` and `AsyncListenRESTClient`. rg 'from .v1.rest.client import ListenRESTClient' rg 'from .v1.rest.async_client import AsyncListenRESTClient'Length of output: 114
Script:
#!/bin/bash # Description: List the contents of the listen module to verify the presence of re-exports for `ListenRESTClient` and `AsyncListenRESTClient`. # Test: List the contents of the listen module. Expect: Verification of re-exports for `ListenRESTClient` and `AsyncListenRESTClient`. fd -t f listen | xargs catLength of output: 8467
Tools
Ruff
40-40:
.listen.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
40-40:
.listen.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
106-107: Verify the updated imports for SpeakRESTOptions and SpeakWebSocketOptions.Ensure that the imports for
SpeakRESTOptionsandSpeakWebSocketOptionsare correct and consistent with the rest of the codebase.Tools
Ruff
106-106:
.speak.SpeakRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
107-107:
.speak.SpeakWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/__init__.py (3)
67-67: Verify the updated imports for ListenRESTClient and AsyncListenRESTClient.Ensure that the imports for
ListenRESTClientandAsyncListenRESTClientare correct and consistent with the rest of the codebase.Verification successful
Verified the updated imports for
ListenRESTClientandAsyncListenRESTClient.The imports for
ListenRESTClientandAsyncListenRESTClientare correctly and consistently used in the codebase.
deepgram/__init__.py:from .client import ListenRESTClient, AsyncListenRESTClientScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all import statements for `ListenRESTClient` and `AsyncListenRESTClient` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --type python -A 5 $'from .client import ListenRESTClient, AsyncListenRESTClient'Length of output: 116
Script:
#!/bin/bash # Description: Verify all import statements for `ListenRESTClient` and `AsyncListenRESTClient` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --type py -A 5 $'from .client import ListenRESTClient, AsyncListenRESTClient'Length of output: 388
Tools
Ruff
67-67:
.client.ListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
67-67:
.client.AsyncListenRESTClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
50-51: Verify the updated imports for LiveClient and AsyncLiveClient.Ensure that the imports for
LiveClientandAsyncLiveClientare correct and consistent with the rest of the codebase.Verification successful
The imports for
LiveClientandAsyncLiveClientare correctly updated and consistent across the codebase.
- The import statements are present in
deepgram/__init__.py.- Multiple occurrences in other files confirm their usage and correctness.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all import statements for `LiveClient` and `AsyncLiveClient` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --type python -A 5 $'from .client import LiveClient, AsyncLiveClient'Length of output: 104
Script:
#!/bin/bash # Description: Verify all import statements for `LiveClient` and `AsyncLiveClient` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg 'from .client import LiveClient, AsyncLiveClient' -A 5Length of output: 464
Script:
#!/bin/bash # Description: Verify the usage of `LiveClient` and `AsyncLiveClient` in the codebase. # Test: Search for the usage of `LiveClient` and `AsyncLiveClient` in the codebase. rg 'LiveClient' rg 'AsyncLiveClient'Length of output: 3630
Tools
Ruff
50-50:
.client.LiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
50-50:
.client.AsyncLiveClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
51-51:
.client.ListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
51-51:
.client.AsyncListenWebSocketClientimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
97-98: Verify the updated imports for SpeakRESTOptions and SpeakWebSocketOptions.Ensure that the imports for
SpeakRESTOptionsandSpeakWebSocketOptionsare correct and consistent with the rest of the codebase.Verification successful
Verified the updated imports for
SpeakRESTOptionsandSpeakWebSocketOptionsThe imports for
SpeakRESTOptionsandSpeakWebSocketOptionsare correctly and consistently used throughout the codebase.
deepgram/__init__.pydeepgram/client.pydeepgram/clients/speak/client.pydeepgram/clients/speak/v1/rest/async_client.pydeepgram/clients/speak/v1/websocket/client.pyScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all import statements for `SpeakRESTOptions` and `SpeakWebSocketOptions` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --type python -A 5 $'from .client import SpeakRESTOptions, SpeakWebSocketOptions'Length of output: 116
Script:
#!/bin/bash # Description: Verify all import statements for `SpeakRESTOptions` and `SpeakWebSocketOptions` in the codebase. # Test: Search for the import usage in the codebase. Expect: Only occurrences of the new imports. rg --type py -A 5 $'from .client import SpeakRESTOptions, SpeakWebSocketOptions'Length of output: 81
Script:
#!/bin/bash # Description: Verify the import statements for `SpeakRESTOptions` and `SpeakWebSocketOptions` individually in the codebase. # Test: Search for the import usage of `SpeakRESTOptions` in the codebase. rg --type py -A 5 $'from .client import SpeakRESTOptions' # Test: Search for the import usage of `SpeakWebSocketOptions` in the codebase. rg --type py -A 5 $'from .client import SpeakWebSocketOptions'Length of output: 122
Script:
#!/bin/bash # Description: Locate the definitions of `SpeakRESTOptions` and `SpeakWebSocketOptions` in the codebase. # Test: Search for the definition of `SpeakRESTOptions` in the codebase. rg --type py 'class SpeakRESTOptions' # Test: Search for the definition of `SpeakWebSocketOptions` in the codebase. rg --type py 'class SpeakWebSocketOptions'Length of output: 84
Script:
#!/bin/bash # Description: Locate any occurrences of `SpeakRESTOptions` and `SpeakWebSocketOptions` in the codebase. # Test: Search for any occurrences of `SpeakRESTOptions` in the codebase. rg --type py 'SpeakRESTOptions' # Test: Search for any occurrences of `SpeakWebSocketOptions` in the codebase. rg --type py 'SpeakWebSocketOptions'Length of output: 4125
Tools
Ruff
97-97:
.client.SpeakRESTOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
98-98:
.client.SpeakWebSocketOptionsimported but unused; consider removing, adding to__all__, or using a redundant alias(F401)
deepgram/clients/speak/v1/rest/client.py (4)
17-17: Approved: Added imports look good.The imports for
SpeakRESTOptionsandFileSourceare appropriate and necessary for the functionality provided in this file.
Line range hint
147-168: LGTM! Verify the function usage in the codebase.The changes to the
savefunction are correct and well-implemented. Ensure that all calls tosavematch the updated parameters.Verification successful
LGTM! Verify the function usage in the codebase.
The changes to the
savefunction are correct and well-implemented. All calls tosavematch the updated parameters.
deepgram/clients/speak/v1/rest/client.pydeepgram/clients/speak/v1/rest/async_client.py- Example scripts in the
examples/text-to-speechdirectoryScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all function calls to `save` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg --type python -A 5 $'save'Length of output: 59
Script:
#!/bin/bash # Description: Verify all function calls to `save` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg --type py -A 5 $'save'Length of output: 9688
Line range hint
126-132: LGTM! Verify the function usage in the codebase.The changes to the
filefunction are correct and well-implemented. Ensure that all calls tofilematch the updated parameters.Verification successful
LGTM! No issues found with the
filefunction usage.The
filefunction is defined correctly, and there are no calls to it in the codebase that need verification for parameter usage.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all function calls to `file` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg --type python -A 5 $'file'Length of output: 59
Script:
#!/bin/bash # Description: Verify all function calls to `file` match the updated parameters. # Test: Search for the function definition and its calls in the codebase. rg 'def file' rg 'file('Length of output: 217
Script:
#!/bin/bash # Description: Verify all function calls to `file` match the updated parameters. # Test: Search for the function definition and its calls in the codebase. rg 'def file' rg 'file\('Length of output: 1783
Script:
#!/bin/bash # Description: Verify all function calls to `file` in `deepgram/clients/speak/v1/rest/client.py` match the updated parameters. # Test: Search for the function calls in the specific file. rg 'file\(' deepgram/clients/speak/v1/rest/client.pyLength of output: 107
Script:
#!/bin/bash # Description: Verify all function calls to `file` in the codebase match the updated parameters. # Test: Search for the function calls in the entire codebase. rg 'file\('Length of output: 1639
Line range hint
40-81: LGTM! Verify the function usage in the codebase.The changes to the
streamfunction are correct and well-implemented. Ensure that all calls tostreammatch the updated parameters.deepgram/clients/speak/v1/rest/async_client.py (4)
18-18: Approved: Added imports look good.The imports for
SpeakRESTOptionsandFileSourceare appropriate and necessary for the functionality provided in this file.
Line range hint
126-132: LGTM! Verify the function usage in the codebase.The changes to the
filefunction are correct and well-implemented. Ensure that all calls tofilematch the updated parameters.Verification successful
Function usage verification complete!
The
filefunction indeepgram/clients/speak/v1/rest/async_client.pyis not called externally, ensuring that the updated parameters will not affect other parts of the codebase.
- The changes to the
filefunction are correct and well-implemented.- No external calls to the
filefunction were found in the codebase.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all function calls to `file` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg --type python -A 5 $'file'Length of output: 59
Script:
#!/bin/bash # Description: Verify all function calls to `file` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg --type py -A 5 $'file'Length of output: 120905
Line range hint
147-168: LGTM! Verify the function usage in the codebase.The changes to the
savefunction are correct and well-implemented. Ensure that all calls tosavematch the updated parameters.Verification successful
LGTM! Verify the function usage in the codebase.
The changes to the
savefunction are correct and well-implemented. All calls tosavematch the updated parameters.
examples/text-to-speech/rest/file/woodchuck/main.pyexamples/text-to-speech/rest/file/hello_world/main.pyexamples/text-to-speech/rest/file/legacy_dict_hello_world/main.pyexamples/text-to-speech/rest/file/async_hello_world/main.pydeepgram/clients/speak/v1/rest/async_client.pydeepgram/clients/speak/v1/rest/client.pyScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all function calls to `save` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg --type python -A 5 $'save'Length of output: 59
Script:
#!/bin/bash # Description: Verify all function calls to `save` match the updated parameters. # Test: Search for the function usage. Expect: Only occurrences of the new parameters. rg --type py -A 5 $'save'Length of output: 9688
Line range hint
41-82: LGTM! Verify the function usage in the codebase.The changes to the
streamfunction are correct and well-implemented. Ensure that all calls tostreammatch the updated parameters.deepgram/client.py (1)
36-44: Unused imports should be reviewed.The static analysis tool flagged several imports as unused. Verify if these imports are necessary for backward compatibility or if they can be safely removed.
Verification successful
To proceed with verifying the necessity of the flagged imports, we need to adjust the file type detection in the
rgcommand. Let's re-run the script with the correct file type for Python files.
The flagged imports are necessary for backward compatibility and are used across multiple files. They should not be removed.
LiveClientandAsyncLiveClientare used indeepgram/__init__.py,deepgram/client.py,deepgram/clients/listen_router.py, and other files for backward compatibility.ListenRESTClient,AsyncListenRESTClient,ListenWebSocketClient,AsyncListenWebSocketClient, andListenWebSocketOptionsare used in various parts of the codebase, including client initialization and backward compatibility contexts.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify if the flagged imports are necessary for backward compatibility. # Test: Search for the usage of the flagged imports. Expect: Determine if they are used or can be removed. rg --type python -A 5 $'LiveClient|AsyncLiveClient|ListenRESTClient|AsyncListenRESTClient|ListenWebSocketClient|AsyncListenWebSocketClient|ListenWebSocketOptions|ListenRESTOptions|SpeakRESTOptions|SpeakWebSocketOptions'Length of output: 251
Script:
#!/bin/bash # Description: Verify if the flagged imports are necessary for backward compatibility. # Test: Search for the usage of the flagged imports. Expect: Determine if they are used or can be removed. rg --type py -A 5 $'LiveClient|AsyncLiveClient|ListenRESTClient|AsyncListenRESTClient|ListenWebSocketClient|AsyncListenWebSocketClient|ListenWebSocketOptions|ListenRESTOptions|SpeakRESTOptions|SpeakWebSocketOptions'Length of output: 133978
Tools
Ruff
36-36:
.clients.LiveClientimported but unusedRemove unused import
(F401)
36-36:
.clients.AsyncLiveClientimported but unusedRemove unused import
(F401)
38-38:
.clients.ListenRESTClientimported but unusedRemove unused import
(F401)
39-39:
.clients.AsyncListenRESTClientimported but unusedRemove unused import
(F401)
40-40:
.clients.ListenWebSocketClientimported but unusedRemove unused import
(F401)
41-41:
.clients.AsyncListenWebSocketClientimported but unusedRemove unused import
(F401)
44-44:
.clients.ListenWebSocketOptionsimported but unusedRemove unused import
(F401)
deepgram/clients/speak/v1/websocket/client.py (1)
29-29: Approved: Added import looks good.The import for
SpeakWebSocketOptionsis appropriate and necessary for the functionality provided in this file.deepgram/clients/speak/v1/websocket/async_client.py (1)
28-28: Import Addition ApprovedThe import of
SpeakWebSocketOptionsis necessary for thestartmethod.
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.
Actionable comments posted: 1
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (24)
- deepgram/client.py (2 hunks)
- examples/advanced/rest/direct_invocation/main.py (2 hunks)
- examples/advanced/websocket/direct_invocation/main.py (2 hunks)
- examples/advanced/websocket/microphone_inheritance/main.py (2 hunks)
- examples/speech-to-text/rest/async_url/main.py (1 hunks)
- examples/speech-to-text/rest/callback/callback/main.py (2 hunks)
- examples/speech-to-text/rest/file/main.py (2 hunks)
- examples/speech-to-text/rest/intent/main.py (2 hunks)
- examples/speech-to-text/rest/legacy_dict_url/main.py (1 hunks)
- examples/speech-to-text/rest/sentiment/main.py (2 hunks)
- examples/speech-to-text/rest/stream_file/main.py (1 hunks)
- examples/speech-to-text/rest/summary/main.py (2 hunks)
- examples/speech-to-text/rest/topic/main.py (2 hunks)
- examples/speech-to-text/rest/url/main.py (1 hunks)
- examples/speech-to-text/websocket/async_http/main.py (1 hunks)
- examples/speech-to-text/websocket/async_microphone/main.py (2 hunks)
- examples/speech-to-text/websocket/http/main.py (1 hunks)
- examples/speech-to-text/websocket/legacy_dict_microphone/main.py (1 hunks)
- examples/speech-to-text/websocket/microphone/main.py (2 hunks)
- examples/text-to-speech/rest/file/async_hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py (1 hunks)
- examples/text-to-speech/rest/file/woodchuck/main.py (1 hunks)
- examples/text-to-speech/websocket/interactive/main.py (1 hunks)
Files skipped from review as they are similar to previous changes (17)
- examples/advanced/rest/direct_invocation/main.py
- examples/advanced/websocket/direct_invocation/main.py
- examples/advanced/websocket/microphone_inheritance/main.py
- examples/speech-to-text/rest/async_url/main.py
- examples/speech-to-text/rest/callback/callback/main.py
- examples/speech-to-text/rest/legacy_dict_url/main.py
- examples/speech-to-text/rest/sentiment/main.py
- examples/speech-to-text/rest/stream_file/main.py
- examples/speech-to-text/rest/summary/main.py
- examples/speech-to-text/rest/topic/main.py
- examples/speech-to-text/rest/url/main.py
- examples/speech-to-text/websocket/async_http/main.py
- examples/speech-to-text/websocket/async_microphone/main.py
- examples/speech-to-text/websocket/http/main.py
- examples/speech-to-text/websocket/legacy_dict_microphone/main.py
- examples/speech-to-text/websocket/microphone/main.py
- examples/text-to-speech/rest/file/legacy_dict_hello_world/main.py
Additional context used
Ruff
deepgram/client.py
65-65:
.clients.ListenRESTClientimported but unusedRemove unused import
(F401)
66-66:
.clients.AsyncListenRESTClientimported but unusedRemove unused import
(F401)
Additional comments not posted (6)
examples/text-to-speech/rest/file/hello_world/main.py (1)
34-34: Update method call to the latest namespace.The method call has been updated to use the latest namespace
deepgram.speak.rest.v("1"). Ensure that this change aligns with the new API structure and that the rest of the code is compatible with this update.examples/text-to-speech/rest/file/async_hello_world/main.py (1)
32-32: Update async method call to the latest namespace.The method call has been updated to use the latest namespace
deepgram.speak.asyncrest.v("1"). Ensure that this change aligns with the new API structure and that the rest of the code is compatible with this update.examples/text-to-speech/rest/file/woodchuck/main.py (1)
34-34: Update method call to the latest namespace.The method call has been updated to use the latest namespace
deepgram.speak.rest.v("1"). Ensure that this change aligns with the new API structure and that the rest of the code is compatible with this update.examples/speech-to-text/rest/intent/main.py (1)
50-50: Update method call to the latest namespace.The method call has been updated to use the latest namespace
deepgram.listen.rest.v("1"). Ensure that this change aligns with the new API structure and that the rest of the code is compatible with this update.examples/speech-to-text/rest/file/main.py (1)
51-51: Verify the function usage in the codebase.Ensure that all function calls to
deepgram.listen.rest.v("1")match the new method signature.Verification successful
Verification successful: The function usage of
deepgram.listen.rest.v("1")is consistent across the codebase.Instances were found in the following files:
examples/speech-to-text/rest/url/main.pyexamples/speech-to-text/rest/intent/main.pyexamples/speech-to-text/rest/summary/main.pyexamples/speech-to-text/rest/stream_file/main.pyexamples/speech-to-text/rest/file/main.pyexamples/speech-to-text/rest/sentiment/main.pyexamples/speech-to-text/rest/topic/main.pyexamples/speech-to-text/rest/legacy_dict_url/main.pyexamples/speech-to-text/rest/callback/callback/main.pyThese findings confirm that the updated method signature is used consistently across the codebase.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all function calls to `deepgram.listen.rest.v("1")` match the new signature. # Test: Search for the function usage. Expect: Only occurrences of the new signature. rg --type python -A 5 $'deepgram.listen.rest.v("1")'Length of output: 84
Script:
#!/bin/bash # Description: Verify all function calls to `deepgram.listen.rest.v("1")` match the new signature. # Test: Search for the function usage. Expect: Only occurrences of the new signature. rg --type py -A 5 $'deepgram.listen.rest.v("1")'Length of output: 49
Script:
#!/bin/bash # Description: Verify all function calls to `deepgram.listen.rest` in the codebase. # Test: Search for all instances of `deepgram.listen.rest` to ensure no other similar calls exist. rg --type py -A 5 $'deepgram.listen.rest'Length of output: 5781
examples/text-to-speech/websocket/interactive/main.py (1)
27-27: Verify the logging level change.Ensure that the updated logging level
verboselogs.DEBUGis appropriate for the use case.Verification successful
The logging level change to
verboselogs.DEBUGhas been verified.The
DeepgramClientOptionsinexamples/text-to-speech/websocket/interactive/main.pyis correctly set toverboselogs.DEBUG. Other configurations in the codebase have varying logging levels, but the specific change in the review comment is accurate.
examples/text-to-speech/websocket/interactive/main.pyScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the logging level change to `verboselogs.DEBUG`. # Test: Search for the logging level configuration. Expect: Only occurrences of `verboselogs.DEBUG`. rg --type python -A 5 $'DeepgramClientOptions(verbose=verboselogs.DEBUG)'Length of output: 105
Script:
#!/bin/bash # Description: Verify the logging level change to `verboselogs.DEBUG`. # Test: Search for the logging level configuration. Expect: Only occurrences of `verboselogs.DEBUG`. rg --type py -A 5 $'DeepgramClientOptions(verbose=verboselogs.DEBUG)'Length of output: 70
Script:
#!/bin/bash # Description: Search for all occurrences of `DeepgramClientOptions` in the codebase. # Test: Search for the `DeepgramClientOptions` configuration. rg --type py 'DeepgramClientOptions'Length of output: 15643
Proposed changes
Depends on: #431
This updates all the examples to move off the legacy class names to the newest classes. Exercised all examples to make sure they work.
Types of changes
What types of changes does your code introduce to the community Python SDK?
Put an
xin the boxes that applyChecklist
Put an
xin 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
New Features
Improvements
Refactor
Bug Fixes