-
Notifications
You must be signed in to change notification settings - Fork 260
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
Explosion blunt AP ratio, damage and AP falloff tweaks #3093
Explosion blunt AP ratio, damage and AP falloff tweaks #3093
Conversation
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-8722286657.zip |
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9059249954.zip |
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9069194875.zip |
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9176485998.zip |
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9205016205.zip |
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9374231752.zip |
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9640292589.zip |
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.
After discussing the PR with Perkins, I removed the changes to ExplosionCE.cs, as they were either breaking the underlying physics/geometry at work (e.g energy of explosions going linearly outward in the x-y plane, or getting more powerful the farther they traveled), or were obviously going to be unbalanced (massively increasing the damage/AP when explosions merge.)
Also reduced the 300% increase to Blunt AP to a far more modest 20% increase, and adjusted the default armor pen accordingly. This will boost the blast AP of some of the smaller bombs against armor targets, without completely trivializing armor against bombs (armor generally doesn't get a ton of Blunt AP, compared to the damage of a given explosion).
Kept the Secondary damage AP values.
Ultimately, the larger changes to explosive performance is going to come as part a broader overhaul that is part of what Perkins is working on--we're not going to see a major paradigm shift from just playing with these values.
Going to do a bit of further playtesting and see how it feels, then will probably approve.
bombs did not get bigger
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9640875676.zip |
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9643126804.zip |
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-9674960799.zip |
92d035e
into
CombatExtended-Continued:Development
Changes
<damageAmountBase>800</damageAmountBase>
node to antigrain shells.1-x^2
instead of1-x^0.333
and1-x^0.55
respectively.Reasoning
Alternatives
Testing
Check tests you have performed: