Irritants (Smoke) - New Component#11220
Conversation
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
|
IMO a bit confusing to have a "smoke" addon that does not change the particles, though I can't think of a better name right now. |
Should it go in the goggles addon where eyewear is already providing protection from other things? |
rautamiekka
left a comment
There was a problem hiding this comment.
- Missing macros.
- Inconsistent
*_protectionspellings.
I agree that smoke is maybe not the best, but I also couldn't think of anything else that'd be better I don't think this should go in goggles, just because while they are related to face wear, that's about it. There is no overlap in config, settings, or functionality. |
|
Solved |
|
isInSmoke cost varies based on nearby objects I think it would be better to run this at a reduced interval instead of everyframe |
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
|
This has been working well for us, I think most false positives will be dealt with, but the actual time spent using it is still pretty low, just a few hours. It is possible there could still be false positives, where the player gets themself into some positioning with terrain, grass, etc. that causes it to think they're in smoke and apply the affect. |
|
Immersion cigs rewrite has cough sounds from the dayz mod uner apl-sa. Coughing sound effects: The code for it can be found under core/fnc/adv_fatigue There's even something in there that alerts enemies of you presense if thats of interest. |
|
Maybe I missed it when scanning over the code, but is there an API way to add "exposure" to a unit? I would prefer to avoid having to overwrite the actual value each frame but if i could add a baseline that would be great. Maybe a similar system like the dutyFactor of advanced_fatigue? |
|
I still don't love calling this addon |
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
|
Has problems with tall grass on CUP maps |
|
I had an idea to scan for nearby ammo to determine the type of smoke could also scan at minimum of 3m above ground level, that should clear most foliage |
Ammo can have config event handlers. https://community.bistudio.com/wiki/Arma_3:_Event_Handlers#Ammo_Config_Events |
When merged this pull request will:
blockAIVisibilityhemtt launchfor ADTConsiderations:
IMPORTANT
Component - Add|Fix|Improve|Change|Make|Remove {changes}.