magey / classic-warrior

109 stars 9 forks source link

Instant attack normalization uses wrong multipliers #20

Closed magey closed 5 years ago

magey commented 5 years ago

Extensive testing on the AV beta realm (build 30786) has revealed a critical bug with instant attack normalization. It seems that all weapon types (including ranged weapons) are using Legion normalization multipliers instead of vanilla melee and ranged ones. Here they are for reference:

Weapon type Vanilla multiplier Legion multiplier
Two-handed 3.3 3.6
One-handed 2.4 2.6
Dagger 1.7 1.8
Ranged 2.8 3.0

Test results

Note: all calculations done on unmitigated damage numbers from the combat log, all characters used are level 58 templates

Player Class Attack power Weapon type Speed Damage Log
insec Warrior 692 2H Axe 3.80 46-70 link
insec Warrior 692 2H Sword 2.80 115-173 link
Towneh Warrior 692 2H Axe 3.80 46-70 link
dmzor Rogue 565 1H Sword 2.20 10-20 link
dmzor Rogue 567 Dagger 1.60 1-3 link
dmzor Hunter 622 Crossbow 2.70 20-20 link

insec (2H Axe)

insec (2H Sword)

Towneh (2H Axe)

dmzor (1H Sword)

dmzor (Dagger)

dmzor (Crossbow, 16.5 DPS arrows)

Scyllaqt commented 5 years ago

Normalization is now fixed