Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Listening Outpost Damaged Smoke Effect Sticks After Repair #261

Merged
merged 3 commits into from
Sep 9, 2021

Conversation

commy2
Copy link
Collaborator

@commy2 commy2 commented Sep 9, 2021

ZH 1.04

  • Once a Listening Outpost was in its damaged visual state, it will continuously spawn smoke particles even after it has been fully repaired.
  • Additionally, these smoke particles are visible to enemy players even while the Outpost is stealthed.

After patch:

  • The smoke effect will work as intended: It will be removed once the Outpost is repaired. It will therefore no longer reveal hidden repaired Outposts with smoke columns coming out of nowhere.
  • The smoke columns will still be visible while damaged. This happens for all units and buildings.

@commy2 commy2 added the Bug Something is not working right label Sep 9, 2021
@xezon xezon self-requested a review September 9, 2021 19:22
Copy link
Collaborator

@xezon xezon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

W3D files are no longer automatically added to a .big file by the scripts. We need to explicitly specify their target in either

GeneralsGamePatch\Patch104pZH\Scripts\MAKE_Patch104pArtZH.bat
or
GeneralsGamePatch\Patch104pZH\Scripts\MAKE_Patch104pZH.bat (Core)

If the *.W3D files are optional, then you can put it into the first script.

@xezon
Copy link
Collaborator

xezon commented Sep 9, 2021

What is changed in the W3D files?

@commy2
Copy link
Collaborator Author

commy2 commented Sep 9, 2021

What is changed in the W3D files?

SMOKExx changed to SMOKESxx

The "S" is for "small". The current system is normally used for buildings. It does not work properly for units: When repaired, the effect stays.

In the damaged model I also added SMOKES03 at the exact voxel position where it is used in the normal model. Makes no sense that less smoke is used in the damaged model.

I also changed FIES01 to FIRES01, because it's a typo. With that change, a small fire is added to the center of the bigger smoke column, which looks neat if you notice it.

@xezon
Copy link
Collaborator

xezon commented Sep 9, 2021

Hmm ok, I am not sure what these names refer to. Be aware that changing Bone names can or will cause version mismatch. W3D files that affect client compatibility need to go into the Patch104pZH.big (Core).

@commy2
Copy link
Collaborator Author

commy2 commented Sep 9, 2021

I am not sure what these names refer to.

SMOKExx is just a bone that can be used in the TransitionDamageFX module. Arbitrary name.

SMOKESxx, SMOKEMxx, SMOKELxx, FIRESxx, FIREMxx, FIRELxx are linked in GameData.ini to automatically apply smoke and fire effects to damaged units. They are the names used by every other unit model in the game.

Be aware that changing Bone names can or will cause version mismatch.

This is true, however, this is not an optional model. Is this not the right .bat script? Because the Avenger fix should also be a mandatory model (I see no advantage in somebody staying with the broken canon-wheel thing).

@xezon
Copy link
Collaborator

xezon commented Sep 9, 2021

If it is not an optional Art change, it must go into MAKE_Patch104pZH.bat.
If Avengers are not optional either, then they also must go into MAKE_Patch104pZH.bat.

An optional change is if it is just visual or audio, that causes no version mismatch between clients and does not block any functionality.

@xezon
Copy link
Collaborator

xezon commented Aug 11, 2022

Removing Controversial tag because it is not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working right China Affects China faction Minor Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants