Skip to content

Pr/configurable turn options#45

Open
raghavm243512 wants to merge 11 commits intomainfrom
pr/configurable_turn_options
Open

Pr/configurable turn options#45
raghavm243512 wants to merge 11 commits intomainfrom
pr/configurable_turn_options

Conversation

@raghavm243512
Copy link
Copy Markdown
Collaborator

@raghavm243512 raghavm243512 commented Apr 6, 2026

Few changes:

Fully configurable VAD and turn taking strategies
fix API reading bug
fix API key redaction

Comment thread docs/experiment_setup.md
Base automatically changed from develop/0.1.2 to main April 8, 2026 17:09
@raghavm243512 raghavm243512 marked this pull request as ready for review April 15, 2026 22:37
@tara-servicenow
Copy link
Copy Markdown
Collaborator

Can we add some tests for this logic? And can you also share some test runs on debug more with the different configs

@raghavm243512 raghavm243512 force-pushed the pr/configurable_turn_options branch from 38d4715 to bb2200b Compare April 17, 2026 22:56
@raghavm243512 raghavm243512 force-pushed the pr/configurable_turn_options branch from 8b345a4 to 60f5be5 Compare April 21, 2026 20:24
@raghavm243512
Copy link
Copy Markdown
Collaborator Author

@tara-servicenow

added debug runs with turn configs under snow.core_llm.voice_agent/tree/francis_eval/debug
Don't think I can send full link on public comment

auto-merge was automatically disabled April 21, 2026 21:37

Pull request was closed

@raghavm243512 raghavm243512 reopened this Apr 21, 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.

3 participants