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

Mirror "Spatial Sound Tracking" [WIP] #369

Closed
wants to merge 6 commits into from

Conversation

Doubleumc
Copy link
Contributor

@Doubleumc Doubleumc commented Aug 11, 2024

About the pull request

Mirror of cmss13-devs/cmss13#6748

Sound position updates as the hearer and source move around.

Can test merge. Needs to be tested at scale, see if the architecture is appropriate for CM's demands.

Don't full merge. While (hopefully) fully functional its current form is meant to be restrained proof-of-concept before integrating other aspects and touching a ton of files. Tracking playing sounds is a powerful tool that opens up a lot of possibilities, if done in a way the server can handle.

Explain why it's good for the game

More immersive sound effects.

Testing Photographs and Procedure

Screenshots & Videos

as of minimal hardcode rework port, test 1:
image
image

as of minimal hardcode rework port, test 2:
image
image

as of minimal hardcode rework port, test 3:
image
image

as of minimal hardcode rework port, test 4:
image
image

Changelog

🆑
soundadd: sounds follow their source
/:cl:

stores sound lengths in a json file instead of hardcoding it, also includes the generator for the file
integrate the echo changes
parsing sounds is now the sound_template's job, some more documentation, echo fixes, misc cleanup
minimum bits needed for soundcode parity
@Doubleumc Doubleumc requested a review from fira as a code owner August 11, 2024 04:29
@github-actions github-actions bot added the Sound label Aug 11, 2024
@Doubleumc Doubleumc added the needs tested testmerge this to see what breaks label Aug 24, 2024
@Doubleumc Doubleumc marked this pull request as draft August 27, 2024 21:50
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Doubleumc Doubleumc closed this Sep 13, 2024
@Doubleumc Doubleumc deleted the mirror-spatial-sound branch September 13, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants