[Warlock] Shadowburn missing Conflagration of Chaos crit damage bonus bug#11411
Open
millanzarreta wants to merge 1 commit intosimulationcraft:midnightfrom
Open
[Warlock] Shadowburn missing Conflagration of Chaos crit damage bonus bug#11411millanzarreta wants to merge 1 commit intosimulationcraft:midnightfrom
millanzarreta wants to merge 1 commit intosimulationcraft:midnightfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Conflagration of Chaos crit bonus damage is not applied to Shadowburn (SB). It is working well for Conflagrate (CF).
This might be happening because when they unified the Conflagration of Chaos buffs from Conflagrate and Shadowburn, they didn't actually create a new buff, but rather modified the CoC CF buff adding the SB stuff to it, and perhaps they forgot to add the SB bonus damage part.
We also took the opportunity to retest that the critical damage bonus granted by this talent is equal to the player’s base critical strike chance (without taking into account effects that increase crit chance for whitelisted spells, such as Devastation, Xalan's Ferocity, or Xalan's Cruelty talents). This was only tested with CF, since it is the only one currently working, but once the SB bug is fixed, the bonus should behave the same way there as well. For example, in the following logs the player’s base crit chance is
21.63%, so the final damage is multiplied by1.2163if the spell is affected by CoC.Logs
21.63%player crit base chance,+6%crit chance to CF and SB (whitelisted spells in Devastation rank2 talent), and+2.76%crit chance to CF from Xalan's Ferocity talent, and+5.80%crit chance to SB from Xalan's Ferocity and Xalan's Cruelty talents.230%crit dmg instead of200%because Ruin rank2 talent