Skip to content

Initial commit to fix continuous streaming issue#16

Merged
geekchick merged 2 commits intomainfrom
add-streaming-timeouts-to-irregular-closures
Mar 16, 2022
Merged

Initial commit to fix continuous streaming issue#16
geekchick merged 2 commits intomainfrom
add-streaming-timeouts-to-irregular-closures

Conversation

@geekchick
Copy link
Copy Markdown
Contributor

@geekchick geekchick commented Mar 15, 2022

No description provided.

@geekchick geekchick added the bug Something isn't working label Mar 15, 2022
@geekchick geekchick requested a review from frumsdotxyz March 15, 2022 14:54
@geekchick geekchick self-assigned this Mar 15, 2022
Copy link
Copy Markdown
Contributor

@frumsdotxyz frumsdotxyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can probably avoid the explicit retry counts etc. entirely! I /think/ the issues we've seen aren't with a connection getting stuck open, they're just with an involuntarily closed socket not being properly detected. So...

Comment thread deepgram/transcription.py Outdated
Comment thread deepgram/transcription.py Outdated
Comment thread deepgram/transcription.py Outdated
Comment thread deepgram/transcription.py Outdated
Comment thread deepgram/transcription.py Outdated
@geekchick geekchick requested a review from frumsdotxyz March 15, 2022 20:52
@geekchick geekchick merged commit 9854ef9 into main Mar 16, 2022
@michaeljolley michaeljolley deleted the add-streaming-timeouts-to-irregular-closures branch October 5, 2022 22:08
lukeocodes added a commit that referenced this pull request Apr 29, 2026
…-ci]

This was a duplicate of AGENTS.md content (its own footer admitted so)
and is internal-facing. The maintenance workflow lives in deepgram/agents
PR #16 now; AGENTS.md in this repo remains as the SDK-specific source
of truth for frozen file lists and validation commands.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants