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

Define oscillator phases for the predefined types. #270

Merged
merged 2 commits into from Feb 21, 2014

Conversation

rtoy
Copy link
Member

@rtoy rtoy commented Nov 8, 2013

Define the oscillator waveform (and hence the phase) for each of the predefined oscillator types.

HTML markup almost certainly needs reviewing in addition to the actual text.

</div>
<div id="oscillatorSquare-OscillatorNode-section" class="section">
<h4 id="oscillatorSquare-OscillatorNode">4.23.3.2 "square"</h3>
The waveform for the square wave oscillator is 1 for 0<= t < &pi; and -1 for -&pi; < t < 0.
Copy link
Member

Choose a reason for hiding this comment

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

s/</</g and also other occurences. Plus spaces around binary operators.

Copy link
Member Author

Choose a reason for hiding this comment

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

Did you mean something else? The subst you list is a no-op. Are you saying I should replace < with the appropriate HTML entity?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, I think github replaced ampersand-g-t-semicolon by <. Sorry about that.

@olivierthereaux
Copy link
Contributor

This PR is the result of a group resolution at the 2013-11-07 teleconference. Please apply/merge whenever convenient.

o Replace < with appropriate HTML entities.
o Add some text to say these are idealized waveforms
o Add link to Oscillator Types section from Oscillator Node Interface
  section.
@olivierthereaux
Copy link
Contributor

Current spec draft addresses this with:

All the basic waveform types are 2π-periodic odd functions starting with a positive slope at time 0
http://padenot.github.io/web-audio-api/#phase-of-basic-waveforms

@cwilso cwilso merged commit 70f46a8 into WebAudio:gh-pages Feb 21, 2014
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.

None yet

4 participants