Scourge Strike --> Blood Presence (fix aplicado)

[nicomayo]

A pedido de kubo.
Me saque trinkets/anillo de icc/sigil/rune of fallen crusader/quede con una sola parte del t9 para que el rango de daño se mantenga siempre, sino seria muy complicado.


Osea que todos los SS deberian pegan menos de 3002 normal y 7138 critico.

Resultados del log: (el txt pesa mas de 300kb y el maximo para el foro es 19.5, el que lo quiere pidalo via PM)



En un nivel 80 con gear por la mitad (por todas las cosas que me saque) el daño máximo que hice fue: 3041 de daño normal y 7180 critico. El dr damage tampoco tiene tanta exactitud. 40 mas de daño no creo que sea porque algo esta andando mal. Lo dejo al criterio de rata.

[Draguen]

De todas formas sigue siendo daño extra, algo raro hay.

Fijate de cambiar de stance, de unholy a blood presence, con la misma cantidad de diseases up (y mismo gear). Si el blood presence double dipeara, deberia ser un aumento del 20% del daño masomenos (15% al total, + un 15% del shadow damage). Si el daño es 15%, tonces ya de por si muestra que no.

Yo puse eso en el primer post, que seguro ni te tomaste la molestia de leer:



Y el incremento del daño es del 15% exacto.

[nicomayo]

Te muesto como hace el dr damage para calcular el daño del scourge strike y por qué creo que no sirve. Para ver esto, entrar a la carpeta del addon/data/deathknight.lua y mirar fila 244 en adelante.

	self.Calculation["Scourge Strike"] = function( calculation, ActiveAuras, Talents )
		calculation.finalMod = calculation.finalMod * calculation.dmgM
		if diseaseCount > 0 then
			--Multiplicative - 3.3.3
			calculation.dmgM = calculation.dmgM * (1 + diseaseCount * 0.12 * calculation.dmgM_Magic * (ActiveAuras["Cinderglacier"] or 1) * (1 + (Talents["Black Ice"] or 0)) * ((self:GetSetAmount( "T8 - Damage" ) >= 4) and 1.2 or 1))
			--Is this a better way of displaying it?
			--calculation.extraDamage = 0
			--calculation.extraAvg = diseaseCount * 0.25 * ((self:GetSetAmount( "T8 - Damage" ) >= 4) and 1.2 or 1)
			--calculation.dmgM_Extra = calculation.dmgM_Extra * calculation.dmgM_Magic * (ActiveAuras["Cinderglacier"] or 1) * (1 + (Talents["Black Ice"] or 0))
		end
		if self:GetSetAmount( "T7 - Damage" ) >= 2 then
			calculation.critPerc = calculation.critPerc + 5
		end
		if self:GetSetAmount( "T10 - Damage" ) >= 2 then
			--TODO: Additive/multiplicative?
			calculation.dmgM_Add = calculation.dmgM_Add + 0.1
		end
	end

Por lo que entendí: no incluye ni desolation ni el escudo oseo (aunque si el T10, que es el mismo efecto pero con distinto número), ni ebon plaguebringer (que vos mismo admitiste que era uno de los efectos que aumentaba el daño).
El mismo programador pregunta: Is this a better way of displaying it? creo que eso te tira cualquier confianza al carajo

[Errabundus]

Te muesto como hace el dr damage para calcular el daño del scourge strike y por qué creo que no sirve. Para ver esto, entrar a la carpeta del addon/data/deathknight.lua y mirar fila 244 en adelante.

	self.Calculation["Scourge Strike"] = function( calculation, ActiveAuras, Talents )
		calculation.finalMod = calculation.finalMod * calculation.dmgM
		if diseaseCount > 0 then
			--Multiplicative - 3.3.3
			calculation.dmgM = calculation.dmgM * (1 + diseaseCount * 0.12 * calculation.dmgM_Magic * (ActiveAuras["Cinderglacier"] or 1) * (1 + (Talents["Black Ice"] or 0)) * ((self:GetSetAmount( "T8 - Damage" ) >= 4) and 1.2 or 1))
			--Is this a better way of displaying it?
			--calculation.extraDamage = 0
			--calculation.extraAvg = diseaseCount * 0.25 * ((self:GetSetAmount( "T8 - Damage" ) >= 4) and 1.2 or 1)
			--calculation.dmgM_Extra = calculation.dmgM_Extra * calculation.dmgM_Magic * (ActiveAuras["Cinderglacier"] or 1) * (1 + (Talents["Black Ice"] or 0))
		end
		if self:GetSetAmount( "T7 - Damage" ) >= 2 then
			calculation.critPerc = calculation.critPerc + 5
		end
		if self:GetSetAmount( "T10 - Damage" ) >= 2 then
			--TODO: Additive/multiplicative?
			calculation.dmgM_Add = calculation.dmgM_Add + 0.1
		end
	end

Por lo que entendí: no incluye ni desolation ni el escudo oseo (aunque si el T10, que es el mismo efecto pero con distinto número), ni ebon plaguebringer (que vos mismo admitiste que era uno de los efectos que aumentaba el daño).
El mismo programador pregunta: Is this a better way of displaying it? creo que eso te tira cualquier confianza al carajo

no entendi... si es por el comentario ese tipo de comentarios son RE comunes en codigo que anda bien

[Draguen]

El ebon plague no es un buff directo al Spell Scourge strike. Por eso no lo vas a encontrar en el codigo relevante a los spells. Lo que hace el ebon plague es aumentar el daño magic sobre el target un 13%, no aumentar el daño de un spell un 13%. Se debe calcular de forma distinta.

De todas formas si probas con el talent, y sin el talent, los valores en el Dr.Damage varian correctamente, entonces si se modifica con, por ejemplo el ebon plague.

Te hago una pregunta, que arma usaste en tu testeo, solo por curiosidad

[nicomayo]

Para errabundus: Además de eso (que ya es bastante dudoso para mi), por qué no incluye desolation ni bone shield en la formula y si el T10? Draguen decia que si un modificador aumenta el daño fisico ya no debería aumentar el daño shadow. El efecto 4P T10 aumenta creo que 3% todos los daños (osea que fisico y shadow). Los otros dos hacen lo mismo pero en distinto numero (5% y 2%).
Para Draguen: lothar edge heroica (sin fallen crusader)

[Draguen]

El tier 10 si no me equivoco, el bonus de 2 partes es:

2) Set: Your Obliterate and Scourge Strike abilities deal 10% increased damage, and your Heart Strike ability deals 7% increased damage.

Osea el tier 10 modifica especificamente al Scourge Strike. Y probablemente ese sea el calculo que toma en cuenta el Dr.Damage.

[nicomayo]

ah error mio. pense que hablaba sobre el de 4 partes

[slogan123]

es que estamos hablando de algo logico el loco escribe un codigo para un addon que puede tener bugs.. como el wow como todo codigo de programacion.. pero aca para ellos el DR.DAMAGE es la "posta" cuando lo escribe un chabon 3ero a blizzard que por ahi no se enrosco en el double dip y en otras cosas no pueden tomar un addon, pero paginas antes para draguen " el dr.damage esta bien NO TIENE BUGS " menos idea de programacion tenes draguen, lo mismo con lo otro que dijistes que el cliente de blizzard dice " no lo afecta ningun spell o nose cuanto ", que tiene que ver ? puede estar bug, si todo estuviese bien por default como esta, estariamos jugando a un clon del OFFI estas tan equivocado desde que empezastes a buscarle mierda al dk, ya te dije hay 500 personas, me enrosque leyendo en la guia de elite que es un post muy parecido a este, donde estan al tanto con los mismos post de GC que linkearon aca, quoteandolos ellos mismos y 150 paginas para adelante nadie habla de un nerf o un parche aplicado, es mas siguen afirmando que double dipea con algunas talents, te lo vuelvo a preguntar vos sabes mas que ellos? que estaban al tanto de lo que GC posteaba en ese momento, y ellos tenian la chance de testear en el offi ?


dejen de nombrar al dr.damage es un codigo como cualquier otro puede tener en ese mismo momento 150 bugs no es confiable

[Draguen]

Nico mira, siendote sincero.

Yo no digo que el scourge strike haya estado pegando bien, reconosco que pegaba poco, pero no creo que deba double dipiar el blood presence. Osea si bien hay fuentes que dicen que si, otras tantas dicen que no : /

Osea le faltan cosas al DK, y ojala se arreglen, me parece que la del blood presence es un fix mal hecho. De hecho yo hace rato vengo defendiendo, por ejemplo, el tema del Anti-Magic Shell y el hecho de que le entran muchos debuff (y no le deberia entrar nada magico, ni curse, ni poison, ni etc).

Asi que no piensen que la tengo contra el Dk.

Con el dr.damage mi sospecha principal surgio cuando estaba testeando y cambie de blood a unholy, y obviamente el daño disminuyo. Y despues de una simple cuenta, note que el daño que aumenta o disminuye con el dr.damage es exactamente 15%, lo cual indica que el addon no toma ese double dip.