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

fixed extra callback calls in while using cue to the soundFiles #449

Merged
merged 2 commits into from
Jun 1, 2020

Conversation

endurance21
Copy link
Collaborator

fixes #443

@endurance21
Copy link
Collaborator Author

endurance21 commented Apr 19, 2020

@therewasaguy please have a look on this pr !

@endurance21
Copy link
Collaborator Author

@montoyamoraga
please have a look on this pr !

Copy link
Member

@therewasaguy therewasaguy left a comment

Choose a reason for hiding this comment

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

looks great! How would you feel about trying to ensure that this feature works by adding a test?

@endurance21
Copy link
Collaborator Author

Sounds great! I will be adding a unit test for the same :)

@endurance21 endurance21 force-pushed the processPeaks branch 3 times, most recently from 152b8d2 to 7579400 Compare May 9, 2020 04:03
@endurance21
Copy link
Collaborator Author

I have added the tests for the same !
Selection_106
@therewasaguy have a look at it ! 🙂

@endurance21 endurance21 requested a review from therewasaguy May 11, 2020 10:05
Copy link
Member

@therewasaguy therewasaguy left a comment

Choose a reason for hiding this comment

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

looks good @endurance21 ! Can you resolve the merge conflicts or lmk if I can help with that!

test/tests/p5.SoundFile.js Show resolved Hide resolved
@endurance21
Copy link
Collaborator Author

looks great to me !

@endurance21 endurance21 merged commit 3992740 into processing:master Jun 1, 2020
therewasaguy added a commit to processing/p5.js that referenced this pull request May 26, 2021
* Fix drywet function bug https://github.com/processing/p5.js-sound/pulls/606
* Fix function name in documentation processing/p5.js-sound#603
* Fix repeat initialization of audioWorklet processing/p5.js-sound#593
* Fix issue with p5.Score that prevented parts from being passed as argument processing/p5.js-sound#543
* Fallback when AudioWorklet buffer array length is zero
 processing/p5.js-sound#542
* Remove extra callbacks for SoundFile cue
 processing/p5.js-sound#449
* Fix abnormal behaviour of audio output in p5.signal example section
  processing/p5.js-sound#476
* Fix error while playing sound through `p5.soundFile.play()` method
 processing/p5.js-sound#542

* Library is re-written in ESM ( ECMASCRIPT MODULES).
* Audio Nodes are written as `Class` style rather than being  written as `function` style.
* Library using features of es6 like `let` , `const` keywords and many other es6+ features.
* Testing architecture has been made more clean and robust.
* All above listed improvements are part of [Divyanshu's](https://github.com/endurance21) GSOC-20 Project , more details can be found in his [wrap-up-post](https://github.com/endurance21/GSOC-20-WrapUp).
* Improvements in pre-commit-hook action  processing/p5.js-sound#574

* Fixed broken anchor tags in p5.Audioln
 processing/p5.js-sound#450
* Fixed typo  in p5.fft documentation
  processing/p5.js-sound#433
* Added documentations for P5.oscillator APIs
 processing/p5.js-sound#420
* Improved documentation of p5.soundloop
  processing/p5.js-sound#437
* Fixed broken links in p5.panner3D documentations
 processing/p5.js-sound#465
* Typo improvements in p5.soundloop reference
 processing/p5.js-sound#557

* renames: outputVolume method and Main class processing/p5.js-sound#610

We have deprecated functionality in an effort to focus the API
* p5.Signal
* p5.SoundFile.processPeaks
therewasaguy added a commit to processing/p5.js that referenced this pull request May 26, 2021
Bug Fixes
* Fix drywet function bug https://github.com/processing/p5.js-sound/pulls/606
* Fix function name in documentation processing/p5.js-sound#603
* Fix repeat initialization of audioWorklet processing/p5.js-sound#593
* Fix issue with p5.Score that prevented parts from being passed as argument processing/p5.js-sound#543
* Fallback when AudioWorklet buffer array length is zero
 processing/p5.js-sound#542
* Remove extra callbacks for SoundFile cue
 processing/p5.js-sound#449
* Fix abnormal behaviour of audio output in p5.signal example section
  processing/p5.js-sound#476
* Fix error while playing sound through `p5.soundFile.play()` method
 processing/p5.js-sound#542

Intrastructure Improvements
* Library is re-written in ESM ( ECMASCRIPT MODULES).
* Audio Nodes are written as `Class` style rather than being  written as `function` style.
* Library using features of es6 like `let` , `const` keywords and many other es6+ features.
* Testing architecture has been made more clean and robust.
* All above listed improvements are part of [Divyanshu's](https://github.com/endurance21) GSOC-20 Project , more details can be found in his [wrap-up-post](https://github.com/endurance21/GSOC-20-WrapUp).
* Improvements in pre-commit-hook action  processing/p5.js-sound#574

Documentation Fixes
* Fixed broken anchor tags in p5.Audioln
 processing/p5.js-sound#450
* Fixed typo  in p5.fft documentation
  processing/p5.js-sound#433
* Added documentations for P5.oscillator APIs
 processing/p5.js-sound#420
* Improved documentation of p5.soundloop
  processing/p5.js-sound#437
* Fixed broken links in p5.panner3D documentations
 processing/p5.js-sound#465
* Typo improvements in p5.soundloop reference
 processing/p5.js-sound#557

Deprecations and Renames
* renames: outputVolume method and Main class processing/p5.js-sound#610
* We have deprecated functionality in an effort to focus the API
  * p5.Signal
  * p5.SoundFile.processPeaks
@therewasaguy therewasaguy mentioned this pull request May 26, 2021
3 tasks
therewasaguy added a commit to processing/p5.js that referenced this pull request May 26, 2021
Bug Fixes
* Fix drywet function bug https://github.com/processing/p5.js-sound/pulls/606
* Fix function name in documentation processing/p5.js-sound#603
* Fix repeat initialization of audioWorklet processing/p5.js-sound#593
* Fix issue with p5.Score that prevented parts from being passed as argument processing/p5.js-sound#543
* Fallback when AudioWorklet buffer array length is zero
 processing/p5.js-sound#542
* Remove extra callbacks for SoundFile cue
 processing/p5.js-sound#449
* Fix abnormal behaviour of audio output in p5.signal example section
  processing/p5.js-sound#476
* Fix error while playing sound through `p5.soundFile.play()` method
 processing/p5.js-sound#542
* Bring back p5.PeakDetect which was missing in 1.0.0

Intrastructure Improvements
* Library is re-written in ESM ( ECMASCRIPT MODULES).
* Audio Nodes are written as `Class` style rather than being  written as `function` style.
* Library using features of es6 like `let` , `const` keywords and many other es6+ features.
* Testing architecture has been made more clean and robust.
* All above listed improvements are part of [Divyanshu's](https://github.com/endurance21) GSOC-20 Project , more details can be found in his [wrap-up-post](https://github.com/endurance21/GSOC-20-WrapUp).
* Improvements in pre-commit-hook action  processing/p5.js-sound#574

Documentation Fixes
* Fixed broken anchor tags in p5.Audioln
 processing/p5.js-sound#450
* Fixed typo  in p5.fft documentation
  processing/p5.js-sound#433
* Added documentations for P5.oscillator APIs
 processing/p5.js-sound#420
* Improved documentation of p5.soundloop
  processing/p5.js-sound#437
* Fixed broken links in p5.panner3D documentations
 processing/p5.js-sound#465
* Typo improvements in p5.soundloop reference
 processing/p5.js-sound#557

Deprecations and Renames
* renames: outputVolume method and Main class processing/p5.js-sound#610
* We have deprecated functionality in an effort to focus the API
  * p5.Signal
  * p5.SoundFile.processPeaks
therewasaguy added a commit to processing/p5.js that referenced this pull request May 30, 2021
Bug Fixes
* Fix drywet function bug https://github.com/processing/p5.js-sound/pulls/606
* Fix function name in documentation processing/p5.js-sound#603
* Fix repeat initialization of audioWorklet processing/p5.js-sound#593
* Fix issue with p5.Score that prevented parts from being passed as argument processing/p5.js-sound#543
* Fallback when AudioWorklet buffer array length is zero
 processing/p5.js-sound#542
* Remove extra callbacks for SoundFile cue
 processing/p5.js-sound#449
* Fix abnormal behaviour of audio output in p5.signal example section
  processing/p5.js-sound#476
* Fix error while playing sound through `p5.soundFile.play()` method
 processing/p5.js-sound#542
* Bring back p5.PeakDetect which was missing in 1.0.0

Intrastructure Improvements
* Library is re-written in ESM ( ECMASCRIPT MODULES).
* Audio Nodes are written as `Class` style rather than being  written as `function` style.
* Library using features of es6 like `let` , `const` keywords and many other es6+ features.
* Testing architecture has been made more clean and robust.
* All above listed improvements are part of [Divyanshu's](https://github.com/endurance21) GSOC-20 Project , more details can be found in his [wrap-up-post](https://github.com/endurance21/GSOC-20-WrapUp).
* Improvements in pre-commit-hook action  processing/p5.js-sound#574

Documentation Fixes
* Fixed broken anchor tags in p5.Audioln
 processing/p5.js-sound#450
* Fixed typo  in p5.fft documentation
  processing/p5.js-sound#433
* Added documentations for P5.oscillator APIs
 processing/p5.js-sound#420
* Improved documentation of p5.soundloop
  processing/p5.js-sound#437
* Fixed broken links in p5.panner3D documentations
 processing/p5.js-sound#465
* Typo improvements in p5.soundloop reference
 processing/p5.js-sound#557

Deprecations and Renames
* renames: outputVolume method and Main class processing/p5.js-sound#610
* We have deprecated functionality in an effort to focus the API
  * p5.Signal
  * p5.SoundFile.processPeaks
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.

callback in addCue is called multiple times per cue time
2 participants