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

Jukebox betterment #412

Merged
merged 3 commits into from
Feb 3, 2024
Merged

Jukebox betterment #412

merged 3 commits into from
Feb 3, 2024

Conversation

MrMelbert
Copy link
Owner

See the TG PR for more information.

comfyorange and others added 2 commits January 30, 2024 19:12
… it fully respects deaf people. (#81135)

- Jukebox is refactored into a datum that the rave visor and the jukebox
uses.

- Jukebox UI is now typescript.

- How the Jukebox delivers sound to players has been rewritten.
- Now it adjusts the sound's position in accordance to where the
listener is.
- This implementation was loosely inspired by that done by Baystation
half a decade ago, so kudos to them.
   - Additionally, being deafened will temporarily mute the jukebox.

- And sorry, in refactoring this I snuck in one tiny feature.
- You can now toggle looping on Jukeboxes to play the song
foreeeverrrrr.

It sounds wayyyyyy better. Overhead isn't even that bad, though it could
be tested on a live server to make sure.
@MrMelbert MrMelbert added the Cherry Picked We're behind on schedule; and this is something we fucking need label Jan 31, 2024
@CRITAWAKETS
Copy link
Contributor

CRITAWAKETS commented Jan 31, 2024

We love hearing Second Reality with positional audio!

also the PR in question because melbert lazy: tgstation/tgstation#81135

@MrMelbert MrMelbert merged commit ab87a76 into master Feb 3, 2024
22 checks passed
@MrMelbert MrMelbert deleted the jukebox_port branch February 3, 2024 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cherry Picked We're behind on schedule; and this is something we fucking need
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants