Skip to content

add per user voice activity support - #21

Open
TomOdellSheetMusic wants to merge 1 commit into
SableClient:livekitfrom
TomOdellSheetMusic:per-user-voice-activity
Open

add per user voice activity support#21
TomOdellSheetMusic wants to merge 1 commit into
SableClient:livekitfrom
TomOdellSheetMusic:per-user-voice-activity

Conversation

@TomOdellSheetMusic

@TomOdellSheetMusic TomOdellSheetMusic commented Aug 12, 2026

Copy link
Copy Markdown

Content

This PR adds per user voice activity support by measuring the audio level of each user, applying a bit of debounce and then passes the list of actively speaking with a widget.

Motivation and context

Voice activity indicators look very nice and can help deciphering who is speaking when you are in a new group.
#9

Screenshots / GIFs

UI Changes were made on the Sable Client Repository. Heres a screenshot for reference:
UserRingSpeaking

Tests

Automatic tests are done with a mock audio track and are written for the audio level measuring, voice activity and every debounce option.

Use of generative AI

observeAudioLevel.test.ts was written entirely by generative AI. Changes in main.ts were largely done with copilot autocomplete.

Additional Information

I was not sure what to do with the copyright header, so I took this header and filled in the information.

TomOdellSheetMusic added a commit to TomOdellSheetMusic/Sable that referenced this pull request Aug 12, 2026
@TomOdellSheetMusic
TomOdellSheetMusic marked this pull request as ready for review August 12, 2026 21:33
@TomOdellSheetMusic TomOdellSheetMusic changed the title adds per user voice activity support add per user voice activity support Aug 12, 2026
@TomOdellSheetMusic
TomOdellSheetMusic marked this pull request as draft August 13, 2026 13:12
@TomOdellSheetMusic
TomOdellSheetMusic marked this pull request as ready for review August 13, 2026 13:15
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