Skip to content

fix(assemblyai): stop forcing continuous_partials=true for U3-Pro models#6401

Merged
u9g merged 1 commit into
mainfrom
assemblyai/remove-continuous-partials-default
Jul 13, 2026
Merged

fix(assemblyai): stop forcing continuous_partials=true for U3-Pro models#6401
u9g merged 1 commit into
mainfrom
assemblyai/remove-continuous-partials-default

Conversation

@u9g

@u9g u9g commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

The plugin defaulted continuous_partials=True for U3-Pro family models, added in #5819 when AssemblyAI's server default was false. AssemblyAI has since flipped the server default to enabled (docs), making the plugin override redundant in the common case — and actively counterproductive when speaker_labels is enabled, where the server deliberately disables continuous partials (along with diarization-tuned silence values) so turns break cleanly at speaker changes, only to have the plugin's explicit continuous_partials=true stomp it.

Changes

  • Remove the forced continuous_partials=True default; an unset value stays NOT_GIVEN and is not sent on the wire, so AssemblyAI's server defaults govern (enabled normally, disabled when speaker_labels is on). Explicit user-passed values are unaffected, including via update_options / mid-session UpdateConfiguration.
  • Update the continuous_partials docstring to describe the current server defaults.
  • Update tests that asserted the old LiveKit-only default to assert the value stays unset.

Not addressed here: the plugin's min_turn_silence=100 latency default similarly overrides the server's diarization-tuned 640/768 ms values when speaker_labels is on — left as a separate decision since it's an intentional latency choice.

AssemblyAI now enables continuous_partials server-side by default, and
deliberately disables it when speaker_labels is on so turns break cleanly
at speaker changes. The plugin's forced true default was redundant in the
common case and overrode the diarization-tuned behavior. Leave the
parameter unset unless explicitly provided.
@u9g u9g requested a review from a team as a code owner July 13, 2026 15:15

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@u9g u9g merged commit 680a3c4 into main Jul 13, 2026
26 checks passed
@u9g u9g deleted the assemblyai/remove-continuous-partials-default branch July 13, 2026 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants