-
Notifications
You must be signed in to change notification settings - Fork 740
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
#9027 Gives back ability to put AI unconscious by shooting at limbs #9100
Conversation
…inUnconsciousChance setting. Set to 0 to disable.
How does changing a setting and a magic var's default values fix the underlying issue you're describing? |
The purpose of this constant is to be at the primary level. Other variables are reffering to this constant in order to have a functionnal logical system. |
Changing the setting default doesn't make much sense, virtually nobody runs ACE as default. And if they do raising the default after this long will just bring people asking questions about why it's happening more now. |
Fair enough for the change on the constant. I wouldn't change the setting's default value, though. |
I still don't understand why, but that was intended to hide and hardcode value. Read comments. |
Oh wow I did not see that, but yes same matters. At the time, I guess 0.5 was a reference value. Which is the goal of constants. At the time you suggested adding the slider, I guess I was still okay with 0.5 default value but since february. What was 0.5 is now 0.35 according to my "damages" tests and preferences. It is an easy and good solution. |
Most firearm hits to limbs will already cause pain near 1 due to low limb armor. The only scenario where this isn't the case is against armored uniforms. Changing the constant's value will do absolutely nothing. Increase the chance for unconsciousness from pain (or the pain coefficient) if you want units to go unconscious more often. |
The problem is that it also affects players as much as AI. So you would suggest having both unconscious chance for AI and players. But now if you merged the slider for painUnconsciousThreshold. It will allow armored AI to be killed with bullets by shooting at limbs only. Specifally requested for sniping with 12.7. It was too much frustrating seeing that x3 12.7 shot on a leg is not enough. Adding the slider for the treshold will help. |
Fix related to #9027
When merged this pull request will:
ace_medical_const_painUnconscious
to 0.35 (from 0.5).painUnconsciousChance
to 0.3 (from 0.1) to add a little dose of variation.TIP: If you want your AI to keep being limbs insensitive:
ace_medical_fatalDamageSource
to 0 (head and so called "heart") which is exactly the goal of that setting already.IMPORTANT
Component - Add|Fix|Improve|Change|Make|Remove {changes}
.