Skip to content

Remove end_of_turn_confidence_threshold from turn detection guide#863

Open
dylan-duan-aai wants to merge 3 commits intomainfrom
docs/update-turn-detection-guide
Open

Remove end_of_turn_confidence_threshold from turn detection guide#863
dylan-duan-aai wants to merge 3 commits intomainfrom
docs/update-turn-detection-guide

Conversation

@dylan-duan-aai
Copy link
Copy Markdown
Contributor

@dylan-duan-aai dylan-duan-aai commented Apr 3, 2026

Summary

  • Removed all 12 lines referencing end_of_turn_confidence_threshold from config presets and print statements
  • This parameter does not exist on U3 Pro (punctuation-based turn detection replaced confidence-based)
  • min_turn_silence and max_turn_silence are kept — those are valid U3 Pro tuning knobs

Test plan

  • Verify no remaining references to end_of_turn_confidence_threshold in the file
  • Confirm min_turn_silence and max_turn_silence are still present in config presets
  • Verify the code examples render correctly on the docs site

Open with Devin

This parameter does not exist on U3 Pro (punctuation-based turn
detection replaced confidence-based). Only min_turn_silence and
max_turn_silence are valid tuning knobs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

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

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

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

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.

1 participant