GeneralsGamePatch icon indicating copy to clipboard operation
GeneralsGamePatch copied to clipboard

Improve the particle effect for destroyed GLA Demo units

Open xezon opened this issue 2 years ago • 12 comments

This change improves the particle effect for destroyed GLA Demo units. It now looks more similar to the regular Demo Suicide explosion, but with slightly smaller effects and without the explosion pillars.

shot_20230830_193613_3

xezon avatar Aug 29 '23 20:08 xezon

Image added. Documentation added. Looks good.

xezon avatar Aug 30 '23 17:08 xezon

I want to try this first. The difference should be very noticable, since it does vastly different amounts of damage. 1.04 already is not very clear about this.

commy2 avatar Aug 30 '23 17:08 commy2

They are distinct, also thanks to the diagonal line lol.

shot_20230830_200223_11

xezon avatar Aug 30 '23 18:08 xezon

Yea, glow helps. Right one looks kind of big though. It does pathetic damage, so it really should just be a puff. The donut adds to the size and probably too much.

commy2 avatar Aug 30 '23 18:08 commy2

It does not deal much damage, but it has huge damage radius. Bigger than suicide radius. Also keep in mind it needs to fit on vehicles. If it is too small, it will look unfitting.

I made a bit smaller now. Check.

xezon avatar Aug 30 '23 18:08 xezon

will do

commy2 avatar Aug 30 '23 19:08 commy2

commy2

Re visuals: I think the donut could be smaller. I mean, same radius obv, but smaller wall thickness. Or maybe the radius could be decreased if it is also decided that the secondary damage radius should be 50 instead of 70.

xezon

We can reduce the radius to match or undercut suicide radius yes. Does not make sense that it is larger than suicide.

Edit:

  • Done with #2420

xezon avatar Jun 08 '24 14:06 xezon

I've noticed that there's a continuous effects performance degradation the longer the game goes vs demo. Some visual & transition effects stop displaying the more demo units you kill & the more explosion they produce. I've tried playing with ParticleSystems Priority - changing from CRITICAL & WEAPON_EXPLOSION to DEATH_EXPLOSION but that didn't solve the issue.

Seems like effects are accumulating and prevent other effects from displaying fully. Similar to when you have too many firestorms simultaneously.

Float1ngFree avatar Jun 14 '24 14:06 Float1ngFree

Can you try to figure out which particle system is causing it? It needs its own bug report. Likewise we know that DisabledEffectBinaryShower0 is causing problems (#1739).

xezon avatar Jun 14 '24 14:06 xezon

Can you try to figure out which particle system is causing it? It needs its own bug report. Likewise we know that DisabledEffectBinaryShower0 is causing problems (#1739).

I've tried switching back to WeaponFX_DemoSuicideDynamitePackDetonationPlusFire but the result is the same. Commented out ExplosionSmall and DemoExplosionSmokeLite -- a bit better but still accumulates. Set SystemLifetime for these new small ParticleSystems similar to WeaponFX_DemoSuicideDynamitePackDetonationPlusFire but no difference.

I'm sorry, I don't have much experience dealing with ParticleSystem items... You could try his on your side, just killing Demo AI units at his base (after demo upgrade) and observe how effects seem to stuck more and more...

BTW, in https://github.com/TheSuperHackers/GeneralsGamePatch/issues/1519 you've turned questionable ParticleSystem into green for debugging -- can you please share how that could be done? I want to debug this as well and check out couple more effects I have my doubts about.

Float1ngFree avatar Jun 14 '24 17:06 Float1ngFree

Is this performance bug Patch specific or in Original game?

xezon avatar Jun 14 '24 17:06 xezon

Is this performance bug Patch specific or in Original game?

Seems to be Patch, original plays all animations no matter how many units/buildings you kill in at demo base...

Float1ngFree avatar Jun 14 '24 18:06 Float1ngFree