-
Notifications
You must be signed in to change notification settings - Fork 27
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
Conversation
There was a problem hiding this 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.
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. |
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 |
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.
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). |
If it is not an optional Art change, it must go into An optional change is if it is just visual or audio, that causes no version mismatch between clients and does not block any functionality. |
Removing Controversial tag because it is not. |
ZH 1.04
After patch: