ACE3 icon indicating copy to clipboard operation
ACE3 copied to clipboard

Remove PhysX from ugl smoke grenades

Open commy2 opened this issue 4 years ago • 3 comments

When merged this pull request will: With default values, the only (apparent) differences between shotSmokeX and shotDeploy are:

  • no smoke creation
  • no smoke sound
  • old physics with configurable bounce (deflectionSlowDown etc.)

shotSmoke has smoke creation, but no smoke sounds, which is why #4648 was rejected.

Best way to achieve old physcis, but with smoke sound was to attach a shotSmokeX to a shotShell/shotSmoke/shotDeploy. Of these shotDeploy is the only one that has no side effects (explosion of shotShell, [duplicate] smoke creation of shotSmoke) with default values. There don't seem to be duplicated bounce sound effects, likely because (most) physics are disabled for attached objects.

Both, the "physics projectile" (shotDeploy) and the "smoke projectile" (shotSmokeX) are deleted after timeToLive.

commy2 avatar Oct 20 '19 09:10 commy2

Doesn't work in mp, probably can't attach to simulation type test

PabstMirror avatar Oct 20 '19 15:10 PabstMirror

It may be that every client has their own projectile, so it needs createVehicleLocal and global execution. I will change it, can you retry then @PabstMirror ?

commy2 avatar Oct 20 '19 15:10 commy2

floating in mid air test2

also, I think i heard multiple "grenade pin" effects from shooters location, not positive (cvl might not be local)

PabstMirror avatar Oct 20 '19 15:10 PabstMirror

We could provide an optional for this. We use @BaerMitUmlaut's version: https://github.com/Theseus-Aegis/Mods/blob/5ce562312c08d9a6d01e046886c29d09f874c991/addons/ballistics/CfgAmmo.hpp

But it's also so simple that any group that would use it probably has their own version of it already.

jonpas avatar Sep 11 '23 00:09 jonpas

We could provide an optional for this. We use @BaerMitUmlaut's version: https://github.com/Theseus-Aegis/Mods/blob/5ce562312c08d9a6d01e046886c29d09f874c991/addons/ballistics/CfgAmmo.hpp

But it's also so simple that any group that would use it probably has their own version of it already.

Accessibility is more important than how simple it is.

rautamiekka avatar Sep 11 '23 23:09 rautamiekka