Skip to content

Conversation

mdydek
Copy link
Contributor

@mdydek mdydek commented Oct 3, 2025

Closes RNAA-204
Closes RNAA-216

⚠️ Breaking changes ⚠️

Introduced changes

  • added test for stereo panner
  • fixed stereo panner behaviour for 1 channel input
  • aligned processNode with web audio api spec. currently it is able to process AudioBus with different number of channels than returned one

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added/Conducted relevant tests
  • Performed self-review of the code
  • Updated Web Audio API coverage
  • Added support for web

@mdydek mdydek added the tests label Oct 3, 2025
@mdydek mdydek self-assigned this Oct 3, 2025
Copy link
Contributor

@poneciak57 poneciak57 left a comment

Choose a reason for hiding this comment

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

Small neatpick. Overally great job.

maciejmakowski2003 added 2 commits October 6, 2025 15:26
@maciejmakowski2003 maciejmakowski2003 merged commit 90bd367 into main Oct 6, 2025
3 checks passed
@maciejmakowski2003 maciejmakowski2003 deleted the test/stereo-panner-test branch October 6, 2025 14:09
vltkv pushed a commit that referenced this pull request Oct 6, 2025
* test: tests for stereo panner

* refactor: aligned processNode to web audio api spec and fixed sterao panner

* test: fixed tests after changing StereoPannerNode implementation

* ci: yarn format

* fix: nitpick

* fix: lint

---------

Co-authored-by: maciejmakowski2003 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants