-
Notifications
You must be signed in to change notification settings - Fork 166
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
Link to correct algorithm for StereoPannerNode, mention algorithm is equal-power #1770
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch! Just a few comments....
index.bs
Outdated
@@ -11506,7 +11506,7 @@ sound. | |||
</figure> | |||
|
|||
<h4 id="stereopanner-algorithm"> | |||
StereoPannerNode Panning</h4> | |||
StereoPannerNode Panning (equal-power only)</h4> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think saying equal-power only adds anything. And nowhere in the algorithm does it say equal-power.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mentioned it because it was mentioned in the link, but I'll remove both
index.bs
Outdated
@@ -9034,7 +9034,7 @@ Attributes</h4> | |||
The {{StereoPannerNode}} Interface</h3> | |||
|
|||
This interface represents a processing node which positions an | |||
incoming audio stream in a stereo image using a low-cost <a href="#Spatialization-equal-power-panning">equal-power panning | |||
incoming audio stream in a stereo image using <a href="#stereopanner-algorithm">a low-cost equal-power panning |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should just leave out "equal-power"? If so, we should also remove "equai-power" from the API Overview for the stereo panner.
StereoPannerNode currently links to PannerNode's algorithm, which asks for the computation of the azimuth. This isn't necessary, StereoPannerNode's `pan` argument is the equivalent of the azimuth, in different units.
205a853
to
113885f
Compare
Updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
StereoPannerNode currently links to PannerNode's algorithm, which asks for the computation of the azimuth. This isn't necessary, StereoPannerNode's
pan
argument is the equivalent of the azimuth, in different units.r? @rtoy
Preview | Diff