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

(OscillatorDetune): Oscillator.detune attribute not defined #147

Closed
olivierthereaux opened this issue Sep 11, 2013 · 3 comments
Closed

(OscillatorDetune): Oscillator.detune attribute not defined #147

olivierthereaux opened this issue Sep 11, 2013 · 3 comments

Comments

@olivierthereaux
Copy link
Contributor

Originally reported on W3C Bugzilla ISSUE-17369 Tue, 05 Jun 2012 11:54:31 GMT
Reported by Philip Jägenstedt
Assigned to

Audio-ISSUE-83 (OscillatorDetune): Oscillator.detune attribute not defined [Web Audio API]

http://www.w3.org/2011/audio/track/issues/83

Raised by: Philip Jägenstedt
On product: Web Audio API

https://dvcs.w3.org/hg/audio/raw-file/tip/webaudio/specification.html#Oscillator

The behavior of the Oscillator detune attribute is not well defined. It seems superfluous, since the base frequency can simply be set to the desired (detuned) frequency instead of being calculated from frequency and detune.

Suggestion: drop the detune attribute.

@olivierthereaux
Copy link
Contributor Author

Original comment by Chris Rogers on W3C Bugzilla. Wed, 10 Oct 2012 00:22:36 GMT

Fixed:
https://dvcs.w3.org/hg/audio/rev/cb14c7445a27

The .detune attribute is important because common synthesis algorithms require controlling both the base frequency and a pitch offset.

@olivierthereaux
Copy link
Contributor Author

Original comment by Philip Jägenstedt on W3C Bugzilla. Fri, 12 Oct 2012 12:42:59 GMT

Question answered by computedFrequency(t) = frequency(t) * pow(2, detune(t) / 1200)

@padenot
Copy link
Member

padenot commented Sep 16, 2013

(This should have been set as closed).

@padenot padenot closed this as completed Sep 16, 2013
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

2 participants