Skip to content

Conversation

@docEdub
Copy link
Contributor

@docEdub docEdub commented Nov 14, 2025

No description provided.

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 14, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@docEdub docEdub force-pushed the 251114-reimplement-old-sound-class branch from 33d1d2f to eb0bcc4 Compare November 17, 2025 21:03
@docEdub docEdub added the audio label Nov 17, 2025
@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 17, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 18, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 24, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 24, 2025

Initial volume is currently `1` when `useCustomAttenuation = true`, which plays the sound for a short amount of time without using the custom attenuation value.

This change sets the initial volume to `0` in this case so the sound is silent before the custom attenuation value kicks in.
@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@docEdub docEdub marked this pull request as ready for review November 25, 2025 16:24
@docEdub docEdub marked this pull request as draft November 25, 2025 16:24
@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

Timing variations are still causing tests using realtime audio context to be flaky. This change expands the acceptable value ranges for them so they are less flaky.
@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 25, 2025

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.

3 participants