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

PCMSound utility functions | makeMono | makeStereo | append #272

Merged
merged 7 commits into from
Nov 9, 2023

Conversation

Raffaello
Copy link
Owner

@Raffaello Raffaello commented Nov 9, 2023

  • concatenate PCMSound to create a new one, this requires to match the format (stereo and freq) of both sounds. Sound1 is the one to keep reference.

  • generate "interrupts"/event when a sound is over.
  • PCMSoundStream could have multiple PCMSound fragments as well.. but not sure if it will make sense....
  • what about a "playlist" functionality? .....

moved to #273

@Raffaello Raffaello self-assigned this Nov 9, 2023
@Raffaello Raffaello changed the title update README PCMSound concat utility function Nov 9, 2023
@Raffaello Raffaello added the enhancement New feature or request label Nov 9, 2023
@Raffaello Raffaello added this to In progress in Digital Sound via automation Nov 9, 2023
@Raffaello Raffaello changed the title PCMSound concat utility function PCMSound utility functions | makeMono | makeStereo | append Nov 9, 2023
Copy link

sonarcloud bot commented Nov 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Raffaello Raffaello merged commit e64d17e into master Nov 9, 2023
8 checks passed
Digital Sound automation moved this from In progress to Done Nov 9, 2023
@Raffaello Raffaello deleted the pcm_audio_concat branch November 9, 2023 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant