Skip to content

Update spec with latest on-device changes #157

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
May 27, 2025
Merged

Conversation

evanbliu
Copy link
Collaborator

@evanbliu evanbliu commented May 7, 2025

Closes #155
Closes #156

This PR contains the following changes:

  • Rename availableOnDevice() to available() and installOnDevice() to install().
  • Update available() and install() parameters to take in a dictionary encapsulating the on-device model options.
  • Add a note on forbidding personalization when recognizing audio from MediaStreamTracks.
  • Remove SpeechRecognitionMode and replace it with SpeechRecognitionOptions.processLocally.
  • Update the install() and available() methods to handle a collection of languages.

Preview | Diff

@evanbliu evanbliu requested a review from padenot May 7, 2025 22:50
@evanbliu evanbliu changed the title Update availableOnDevice and installOnDevice parameters Update spec with latest on-device changes May 16, 2025
@evanbliu
Copy link
Collaborator Author

@padenot - I updated this PR with the rest of the changes we discussed at the Audio WG meeting yesterday. Please take a look when you get a chance :)

evanbliu and others added 5 commits May 19, 2025 11:37
Co-authored-by: Paul Adenot <[email protected]>
Co-authored-by: Paul Adenot <[email protected]>
Co-authored-by: Paul Adenot <[email protected]>
Co-authored-by: Paul Adenot <[email protected]>
Co-authored-by: Paul Adenot <[email protected]>
@evanbliu evanbliu requested a review from padenot May 20, 2025 00:13
@evanbliu
Copy link
Collaborator Author

Gentle ping -- @padenot please take a look when you get a chance. Thanks!

Co-authored-by: Paul Adenot <[email protected]>
@evanbliu
Copy link
Collaborator Author

According to the Web IDL spec, dictionaries can't be attributes, so I removed the options attribute from the SpeechRecognition interface and instead just added a boolean attribute for processLocally. The added benefit of this change is that there can no longer be a state where SpeechRecognition.lang and SpeechRecognition.options.langs have different values.

evanbliu and others added 4 commits May 27, 2025 13:23
Co-authored-by: Paul Adenot <[email protected]>
Co-authored-by: Paul Adenot <[email protected]>
Co-authored-by: Paul Adenot <[email protected]>
Co-authored-by: Paul Adenot <[email protected]>
@evanbliu evanbliu merged commit e8a36c6 into WebAudio:main May 27, 2025
1 check passed
@evanbliu evanbliu deleted the Params branch May 27, 2025 20:40
github-actions bot added a commit that referenced this pull request May 27, 2025
SHA: e8a36c6
Reason: push, by evanbliu

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

Consider renaming SpeechRecognitionMode enum values Consider changing installOnDevice() parameter to InstallOptions
3 participants