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.

Sign up for GitHub

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

Jump to bottom

Oscillator.frequency/detune should link to computedFrequency #1363

Closed
rtoy opened this issue Sep 26, 2017 · 0 comments
Closed

Oscillator.frequency/detune should link to computedFrequency #1363

rtoy opened this issue Sep 26, 2017 · 0 comments
Assignees
@mdjp
Milestone
Web Audio V1

Comments

@rtoy
Copy link
Member

rtoy commented Sep 26, 2017

In https://webaudio.github.io/web-audio-api/#dom-oscillatornode-frequency, it is mentioned that frequency parameter forms a compound parameter with detune. That's fine. But it would be really helpful if this also linked to definition of computedFrequency so that you can easily find out how these are combined and what the nominal range of the computed frequency is.

It's hard to tell what the range is from reading the nominal range of detune (unlimited) and frequency (+/- Nyquist).
@mdjp mdjp added this to the Web Audio V1 milestone Sep 28, 2017
@mdjp mdjp self-assigned this Oct 4, 2017
@rtoy rtoy closed this as completed in 39b2882 Oct 16, 2017
rtoy added a commit that referenced this issue Oct 16, 2017
@rtoy
Merge pull request #1408 from rtoy/1363-osc-link-to-computed-freq
e0b8e4f 
Fix #1363: Add links to Oscillator computed frequency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mdjp mdjp

Labels
None yet
Projects
None yet
Milestone
Web Audio V1
Development

No branches or pull requests
3 participants
@joeberkovitz @rtoy @mdjp