Skip to content
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

fundsp #33

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

fundsp #33

wants to merge 5 commits into from

Conversation

bschwind
Copy link
Member

@bschwind bschwind commented Apr 26, 2023

I'm still playing around with this, but I have a basic proof-of-concept for generating sound and modifying it via atomic variables.

Next step will probably be to re-add the sound samples we had previously with the jukebox.

For playing multiple sound effects (in a potentially spammy manner), we could probably draw some inspiration from how video games do it: https://www.reddit.com/r/gamedev/comments/dq4k3q/how_do_you_handle_a_bunch_of_sound_effects/f60hlly/

Ok(Self { _stream: stream, custom_organ_hz, _frontend_net: frontend_net })
}

fn build_dsp_graph(custom_organ_hz: &Shared<f64>) -> (Net64, Net64Backend) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe it's just lack of experience with sound synthesis, but I'm a bit lost as to what this function does (especially since it uses that strange DSL reusing the shift operators and whatnot) so a couple comments would go a long way here.

Copy link
Contributor

@PabloMansanet PabloMansanet left a comment

Choose a reason for hiding this comment

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

LGTM. This library looks a bit alien to me, but if it's the best out there let's use it.

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.

2 participants