Skip to content

Conversation

@OverlordZorn
Copy link
Contributor

@OverlordZorn OverlordZorn commented Sep 8, 2025

This provides an option to ignore the "disableSoundAttenuation" check per vehicle.

disableSoundAttenuation = 1 is inherited from CargoTurret. This causes all FFV Seats to ignore any configured sound attenuations.

forceSoundAttenuation has been added as a shortcut, to avoid having to edit each individual turret manually.

@OverlordZorn
Copy link
Contributor Author

OverlordZorn commented Sep 8, 2025

@Timi007 Hi. Sorry for the bother but i'd like to ask you regarding some acre attenuation stuff.

https://github.com/IDI-Systems/acre2/blob/master/addons/sys_attenuate/fnc_getAttenuationEffectType.sqf#L30

you're checking the vanilla disableSoundAttenuation and based of this, apply the attenuation effect or not.

Do you by chance remember why you added this check?

@Timi007
Copy link
Contributor

Timi007 commented Sep 8, 2025

@Timi007 Hi. Sorry for the bother but i'd like to ask you regarding some acre attenuation stuff.

https://github.com/IDI-Systems/acre2/blob/master/addons/sys_attenuate/fnc_getAttenuationEffectType.sqf#L30

you're checking the vanilla disableSoundAttenuation and based of this, apply the attenuation effect or not.

Do you by chance remember why you added this check?

I didn't add this check. It was already there in the original code. All I did is move the code into its own function and refactor it a bit. So, I don't know either why this check is there. ¯_(ツ)_/¯

@jonpas
Copy link
Member

jonpas commented Dec 14, 2025

"ignore disable", maybe a better name would be something like enableSoundAttenuationor forceSoundAttenuation?

Documentation would be nice for this too.

@jonpas jonpas added this to the 2.14.0 milestone Dec 14, 2025
@OverlordZorn OverlordZorn changed the title Attenuate - Add ignore_disableSoundAttenuation to vehicle config Attenuate - Add forceSoundAttenuation to vehicle config Jan 13, 2026
OverlordZorn added a commit to OverlordZorn/acre2 that referenced this pull request Jan 13, 2026
@jonpas jonpas merged commit 2fc4055 into IDI-Systems:master Jan 14, 2026
3 checks passed
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.

3 participants