saltyhotdog / BattletechIssueTracker

Public issue tracker to communicate with modders and HBS.
MIT License
6 stars 0 forks source link

Boss Head Called Shot and Grey Head Disease #22

Closed Sheep-y closed 5 years ago

Sheep-y commented 6 years ago

Describe the location in the code your question is about BattleTech.HitLocation method GetHitLocation(Vector3, Mech, float, ArmorLocation, float)

What is the question you have about this code? If target cannot be headshot, the head is removed from the hit table. The head is never added back.

The hit tables are either CombatConstants.HitTables or CombatConstants.GetMechClusterTable and are shared by all hit functions. As a result, after any boss characters are attacked and hit (e.g. when Kamea is hit), all subsequence attacks against anyone from the same direction will never hit the head, until a load game happens or until restart.

Leave any additional comments here The persistent head removal is the reason that boss head is greyed out in called shot panel. If this bug is fixed, called shot will allow selecting the head.

This also means that, if the player try to called shot a boss before the head is removed on hit, the player can actually see and select the head as called shot target.

Workaround: Both bugs are patched in AIM mod v2.0: Grey Head Disease fix, Boss Head Called Shot fix

Sheep-y commented 5 years ago

Issue solved in 1.3 which clone the hit table for every single hit roll. Not ideal in term of speed, but can't argue it works.