namidaka / crpg

Multiplayer mod for Mount & Blade II: Bannerlord
https://c-rpg.eu
GNU General Public License v3.0
14 stars 14 forks source link

mod: polearm bvS Reduction #362

Closed Erdertus closed 2 months ago

Erdertus commented 3 months ago

Reduced polearm BvS damage by 25%.

Changed the final BvS value to be 4.0 for one- and two-handed axes (based on a new weapon class containing only those two), and 3.0 for everything else. Weapons that are not axes do not have a stun-user-on-hit-effect (aka cleave) which makes them less attractive to fight with. Previously this was off set by having axes be the premier BvS weapon over swords. With the introduction of more BvS polearms, axes were further pushed away from being an attractive weapon class, as the code previously only gave weapon class swords a lower BvS value. Now axes receive a higher BvS value, and everything else (including the polearms mentioned) receive the lower value.

This reduces the BvS damage of:

All polearms with BvS The 5 one-handed maces with BvS The 2 two-handed maces with BvS

Muparadzi commented 2 months ago

one thing i just thought about, do we want to add WeaponClass.OneHandedMace to ensure that 1h axes in mace category are not negatively impacted? The downside to that would be that we'd need to look at removing BvS from maces that have it in that category already, which isn't necessarily a downside....

For what reason do we have axes in the mace category? I think the bvs for axes needs to be the best to account for the weird swing mechanics of axes

Erdertus commented 2 months ago

Imo, axes with the hit stun should receive the premier BvS (IE axes) and other weapons, including the maces we have as axes, should receive lower BvS. For visual consistency, if it looks like an axe it shouldn't have some form of BvS imo, but that might just be me ^^

Yeldur commented 2 months ago

Imo, axes with the hit stun should receive the premier BvS (IE axes) and other weapons, including the maces we have as axes, should receive lower BvS. For visual consistency, if it looks like an axe it shouldn't have some form of BvS imo, but that might just be me ^^

Works for me

Yeldur commented 2 months ago

one thing i just thought about, do we want to add WeaponClass.OneHandedMace to ensure that 1h axes in mace category are not negatively impacted? The downside to that would be that we'd need to look at removing BvS from maces that have it in that category already, which isn't necessarily a downside....

For what reason do we have axes in the mace category? I think the bvs for axes needs to be the best to account for the weird swing mechanics of axes

To negate the stop-on-hit mechanic (i.e. to make some axes viable to use as normal weapons)