Skip to content

Add example with WASM audio worklet #3017

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 1 commit into from
Aug 13, 2022

Conversation

lukaslihotzki
Copy link
Contributor

This example runs an oscillator in WASM inside an audio worklet. Volume and frequency is controllable in the UI exposed by the main thread. A design goal was to abstract worklet and JS module handling in a way that can be handled by a library with minimal annoyance for application developers. Therefore, the Rust modules gui, oscillator, wasm_audio, and dependent_module are fairly decoupled.

@alexcrichton
Copy link
Contributor

Thanks! I don't have the time to do an in-depth review of the code here but @Liamolucko would you be perhaps interested in doing so? If not no worries I'm happy to merge regardless.

@lukaslihotzki
Copy link
Contributor Author

An in-depth review would be great if possible.

Note this example would benefit from the feature proposed in #3019. This example could be used as a test case if this feature is implemented.

@lukaslihotzki lukaslihotzki force-pushed the wasm-audio-worklet branch 4 times, most recently from d08f1b6 to b79323e Compare August 5, 2022 10:53
@lukaslihotzki lukaslihotzki force-pushed the wasm-audio-worklet branch 4 times, most recently from c173446 to b553911 Compare August 9, 2022 23:45
Copy link
Collaborator

@Liamolucko Liamolucko left a comment

Choose a reason for hiding this comment

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

Thanks!

@Liamolucko Liamolucko merged commit d881d9d into rustwasm:main Aug 13, 2022
@lukaslihotzki lukaslihotzki deleted the wasm-audio-worklet branch August 13, 2022 18:09
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.

4 participants