webxdc/pixel
16
4
Fork
You've already forked pixel
1

Add gain node for audio #9

Open
link2xt wants to merge 1 commit from link2xt/audio-envelope into main
pull from: link2xt/audio-envelope
merge into: webxdc:main
webxdc:main
webxdc:link2xt/realtime
webxdc:link2xt/buffering
webxdc:link2xt/hold-to-draw
webxdc:hpk/streamlining

Makes this less loud and increases/decreases gain smoothly when update arrives or when there are no more updates.

Makes this less loud and increases/decreases gain smoothly when update arrives or when there are no more updates.
link2xt changed title from (削除) Add gain node for audio (削除ここまで) to WIP: Add gain node for audio 2024年10月21日 19:10:34 +02:00
Author
Owner
Copy link

Oscillator and gain nodes should be made persistent and when a new update arrives if the sound is already playing we should not decrease the value to 0 but keep playing the sound.

What is annoying about the current beeping sound is that it starts and stops all the time.
E.g. try https://tonejs.github.io/examples/envelope and hold the button. It is much less annoying than when you click it repeatedly.

Oscillator and gain nodes should be made persistent and when a new update arrives if the sound is already playing we should not decrease the value to 0 but keep playing the sound. What is annoying about the current beeping sound is that it starts and stops all the time. E.g. try https://tonejs.github.io/examples/envelope and hold the button. It is much less annoying than when you click it repeatedly.
link2xt changed title from (削除) WIP: Add gain node for audio (削除ここまで) to Add gain node for audio 2024年10月21日 19:39:11 +02:00
link2xt force-pushed link2xt/audio-envelope from 2480d38ab8 to f3f74736b4 2024年10月21日 19:43:30 +02:00 Compare
Owner
Copy link

for feedback, i am wondering, if it isn't better to also play a sound when the mouse button is pressed (so when drawing starts, first pixel is sent). currently, it is only played when drawing ends, which make things appear a bit slower and laggy.

maybe one could also play a sound that lasts, say 2 seconds, and fades out (this is probably sustain parameter)

for feedback, i am wondering, if it isn't better to also play a sound when the mouse button is pressed (so when drawing starts, first pixel is sent). currently, it is only played when drawing ends, which make things appear a bit slower and laggy. maybe one could also play a sound that lasts, say 2 seconds, and fades out (this is probably sustain parameter)
Author
Owner
Copy link

for feedback, i am wondering, if it isn't better to also play a sound when the mouse button is pressed (so when drawing starts, first pixel is sent). currently, it is only played when drawing ends, which make things appear a bit slower and laggy.

Sound is played when update is received. We only send update when mouse is released with all the pixels added/removed since mouse was pressed.

> for feedback, i am wondering, if it isn't better to also play a sound when the mouse button is pressed (so when drawing starts, first pixel is sent). currently, it is only played when drawing ends, which make things appear a bit slower and laggy. Sound is played when update is received. We only send update when mouse is released with all the pixels added/removed since mouse was pressed.
hpk approved these changes 2024年10月23日 11:05:48 +02:00
hpk left a comment
Copy link

looks good as far as i can tell.

looks good as far as i can tell.
link2xt force-pushed link2xt/audio-envelope from f3f74736b4 to e7a7f34277 2024年10月24日 18:15:29 +02:00 Compare
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin link2xt/audio-envelope:link2xt/audio-envelope
git switch link2xt/audio-envelope

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff link2xt/audio-envelope
git switch link2xt/audio-envelope
git rebase main
git switch main
git merge --ff-only link2xt/audio-envelope
git switch link2xt/audio-envelope
git rebase main
git switch main
git merge --no-ff link2xt/audio-envelope
git switch main
git merge --squash link2xt/audio-envelope
git switch main
git merge --ff-only link2xt/audio-envelope
git switch main
git merge link2xt/audio-envelope
git push origin main
Sign in to join this conversation.
No reviewers
hpk
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
webxdc/pixel!9
Reference in a new issue
webxdc/pixel
No description provided.
Delete branch "link2xt/audio-envelope"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?