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

sound is broken on Firefox 43.0.4 #11

Open
timaschew opened this issue Feb 14, 2016 · 4 comments
Open

sound is broken on Firefox 43.0.4 #11

timaschew opened this issue Feb 14, 2016 · 4 comments

Comments

@timaschew
Copy link
Contributor

It's playing but there is on each pitch a strange noise.
I'm on OS X 10.11.2

@beku
Copy link

beku commented Oct 16, 2016

Same with Chrome on Android. Very slow tunes come fine. But faster ones show lots of noise.

It sounds like the sound system gets too much requests at the same time and can not handle them properly. It looks like too many initialisations or something around that. Easliy reproducible with musical.js/test/demo/interact.html

My suspection is, that keeping the Audion API busy and alive during playing might fix the bug?

@beku
Copy link

beku commented Oct 16, 2016

Removing setValueAtTime() and linearRampToValueAtTime() appears to eliminate the initial clicks/noice. See
https://bugzilla.mozilla.org/show_bug.cgi?id=1228207
The bug report suggests ambiguities inside the web-audio-api, for handling volume relative to specified time and enabling smooth transitions.

Disabling the above mentioned calls in musical.js produces a sound which is not as expected, but improves much for FF on Android and even Chrome on Android :-/

@beku
Copy link

beku commented Oct 19, 2016

Some more input:
The metronome app in the following tutorial does work as expected. It might be worth to look at how scheduling/look ahead is done there:
https://www.html5rocks.com/en/tutorials/audio/scheduling/
A more complex example from a presentation:
http://webaudiodemos.appspot.com/slides/#/14

@bwl21
Copy link
Contributor

bwl21 commented Nov 14, 2016

I also tried the metronome app in my environment. It really seems to work. Would be glad if musical.js could be updated accordingly.

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

No branches or pull requests

3 participants