Correct Pawn Painshock Threshold Values #3562
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
Reasoning
While
PainShockThreshold
will accept any numerical value, it is clamped between 0.01 and 0.99 by the StatDef, so only values in that range have any discrete function. All values < 0.01 are functionally the same as 0.01, and all values > 0.99 are functionally the same as 0.99--see the min and max values below.Since several Entities have been assigned a threshold value of
90
, there's the distinct impression they were intended to have a lower shock threshold than the Fleshbeasts, which have been set to99
. However, since both these values end up clamped to 0.99, they are mechanically the same in game, making the xml values misleading.Alternatives
Testing
Check tests you have performed: