Skip to content

fix: disable AEC warmup for outbound SIP calls - #6608

Merged
chenghao-mou merged 1 commit into
mainfrom
chenghao/fix/AGT-3199-disable-aec-warmup
Aug 1, 2026
Merged

chenghao-mou merged 1 commit into
mainfrom
chenghao/fix/AGT-3199-disable-aec-warmup

Conversation

@chenghao-mou

@chenghao-mou chenghao-mou commented Jul 29, 2026 •

Copy link
Copy Markdown
Member

Outbound SIP callers can speak before the greeting, but the default AEC warmup suppresses those initial interruptions. Detect outbound SIP participants through the empty sip.ruleID attribute and disable warmup only when the option is omitted; explicit settings and the three-second default for other calls remain unchanged.

Tested with a SIP outbound call.

Fixes AGT-3199

@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 potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

from .transcription.text_transforms import TextTransforms


_SIP_RULE_ID_ATTR = "sip.ruleID"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We really need to have a better abstraction for SIP in general.

@chenghao-mou
chenghao-mou merged commit 76c9fbc into main Aug 1, 2026
58 checks passed
@chenghao-mou
chenghao-mou deleted the chenghao/fix/AGT-3199-disable-aec-warmup branch August 1, 2026 18:46
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.

3 participants