Update Warlock EP/Display stats #108
Conversation
| Stat.StatSpellDamage, | ||
| Stat.StatSpellHit, | ||
| Stat.StatSpellCrit, | ||
| Stat.StatSpellHaste, |
There was a problem hiding this comment.
"Haste" can stay in I believe as other classes also show it? But iirc this is not even a classic stat?
There was a problem hiding this comment.
I don't think we actually show it in the "Classic" codebase looking around at other sims. In SoD we use the cast speed multiplier in place of StatSpellHaste since that stat is effectively ueless when summed up, but in vanilla there wasn't any gear with cast speed modifiers to my knowledge
| Stat.StatShadowPower, | ||
| Stat.StatMP5, | ||
| ], | ||
| epPseudoStats: [PseudoStat.PseudoStatCastSpeedMultiplier], |
There was a problem hiding this comment.
Hm this should work correctly. This is also what we use in SoD to provide the cast speed multiplier (effectively haste that functions differently because haste isn't in classic)
There was a problem hiding this comment.
Oh but you know what cast speed isn't on vanilla gear so that's actually fine
3 participants
Cleans up physical stats from Warlock UI and removes PseudoStatCastSpeedMultiplier ep that breaks stat weight calc modal.