Simple sound physics script

[cyn0x8 (cyn)]

i will try adding that now

@anon53193547 @clustergrowling @PysephDEV as for reverb, im not sure if ill be able to implement functioning reverb with my coding experience, but i will work on it
in the meantime, the github is available for contribution if any of you want to tackle it as well

[IISato (Sato)]

Man I was about to attempt making something like this until this popped up cause of the message from 2 days ago. Really frickin cool.

[ValkyrieHuskyy (DevelopedByHusky)]

Hi there,

There is a huge issue with your script, which is the impact of performance.

I am no experienced scripter to tell you the exact performance-draining elements and functions in the script, but it used up a lot of memory storage and people only received around 16-20 FPS.

For my client, the client memory usage went up to 1.5k

The script itself is useful and I would’ve used it, if not the performance drain.

[cyn0x8 (cyn)]

oh yikes… thank you for showing me that
im not a very experience scripter either, so i might have overlooked some things in the code that might have caused that

im rewriting some of the script right now; ill see if i can find the problem and fix it

[ValkyrieHuskyy (DevelopedByHusky)]

Hi,

I just talked to Shawn, also known as Aotrou.

Both of us investigated my place and found out that future lighting ate most of the performance, so I’m gonna check your script out again and correct myself, if what I said was false.

Here are 2 additional resources regarding the memory leak I experienced:

Future Lighting Technology Causes Memory Leak

I’m starting to use the new Future lighting technology for my game, and it looks stunning. However, we’ve noticed that while using it, there is a massive memory leak. The leak is specifically in the render/system section. We did some testing and here’s what we found: With ShadowMap technology: Immediately after joining: render/system: 14.36 After 5 minutes: render/system: 14.41 With Future technology: Immediately after joining: render/system: 23.61 After 5 minutes: render/system: 49.69 T…

Memory Leak Help? (What is "render/system"?)

took forever to figure out, for anyone else that has this issue: turn off Future lighting I’m not familiar with the procedure to report a bug or anything, but I turned lighting from Future to ShadowMap and it seemed to have solved the issue (as far as I know). Some longer testing will be required to ensure everything’s good but I stopped seeing constant increases when I turned it off. How I determined this: I tried moving everything to a separate place and I had no issues until I turned on …

[cyn0x8 (cyn)]

alrighty

if there are still memory problems afterwards, its likely because it uses value instances to allow for tweening
im going to change the script to lerp attributes instead, because this will use less memory

@WallsAreForClimbing

• Attributes take up less memory on your device
• Attributes are ~18x faster to create dynamically
• Attributes are ~240x faster to delete

[cyn0x8 (cyn)]

a big update to the script is out, along with the testing place!

the script should now be faster and use less memory, but if there are any problems, let me know and i will work on them

changelog

• reorganized script settings for readability
• reorganized code and logic gates to avoid unnecessary declarations/running of code
• fixed some logic errors in the code
• renamed ignore_player_obstruction to ignore_players
• renamed size_threshold to minimum_size
• renamed transparency_threshold to maximum_transparency
• renamed default_eq_values to default values
• added ignore_player_sounds
• added “Water” to the material list (because of a raycast limitation, underwater sounds can only be dampened if the camera is outside of the terrain; you will need to use Region3 or some other method if you want to dampen underwater sounds while the camera is also underwater)
• added lerp_alpha
• added disconnections to player connections
• removed obstruction_value_tween_time
• removed material_obstruction_value_tween_time
• removed value folder and number values in favor of attributes
• removed debounce values, tweens, and tweenservice in favor of lerping