Projekt

Allgemein

Profil

Aktionen

Fehler #35589

geschlossen

[Shaman][Spell] Fire totems and Fire Nova don't seem to scale with shaman crit

Von Pegla vor mehr als 5 Jahren hinzugefügt. Vor fast 5 Jahren aktualisiert.

Status:
Live
Priorität:
Hoch
Zugewiesen an:
-
Kategorie:
Zauber
Zielversion:
Beginn:
12.04.2019
Update Text Deutsch:

Totems profitieren nun von der kritischen Trefferwertung des Schamanen.


Beschreibung

Right now, Searing Totem (https://db.rising-gods.de/?spell=3599), Magma Totem (https://db.rising-gods.de/?spell=8190) and Fire Nova (https://db.rising-gods.de/?spell=1535) have a very low crit chance. With my shammy at 37% spell crit chance, my Fire Nova crits at about 10% while the magma and searing totem probably crit even less. Since Fire Nova is considered a spell casted by the shaman, it should also inherit the spell crit chance from the shaman.

Here you can see the Magma Totem crit fairly often, along with Fire Nova. He looks to be in ToC gear so he probably has about 25-30% crit chance there.

https://www.youtube.com/watch?v=gPho09nW5tE

also in this video you can see the fire nova crit fairly often, probably more than 30% chance, which means it inherited the crit from the shaman:
https://www.youtube.com/watch?v=xOAVlW9Qwns

Aktionen #1

Von vaeterchenfrost vor mehr als 5 Jahren aktualisiert

Confirmed in our logs, Searing&Magma are not crit-buffed (19% crit chance in 68%chance gear).
Also confirmed they should be buffed with player-crit:
https://wowhats.wordpress.com/2010/02/10/fire-dps-totems/
"Stats: our fire totems all benefit from our spell power, hit and intellect; searing and magma also benefit from our crit. Haste does not have any effect on our fire totems"

Aktionen #3

Von vaeterchenfrost vor mehr als 5 Jahren aktualisiert

https://redmine.rising-gods.de/issues/350

Was the first time this problem was adressed, but not correctly implemented.

Firenova seems to work properly right now so only Searing&Magma are not crit-buffed.

Would be nice if there was some feedback on this particular issue, because it is 500-1k dps
for Elemental Shaman ;)

Aktionen #4

Von vaeterchenfrost vor fast 5 Jahren aktualisiert

Just took me 4 hours (too long) to not overlook the fix.
https://github.com/TrinityCore/TrinityCore/pull/4938/commits/b285d7510ff78b01afcc2a55e02f4c6222bfb4dd
Obviously I don't know how the current impl. reads, but looks good.

Learned a lot about Totems on the way...

Aktionen #5

Von Connex vor fast 5 Jahren aktualisiert

  • Status wurde von Bestätigt zu In Bearbeitung geändert
  • Zugewiesen an wurde auf Connex gesetzt

Hab mir das Problem vor n paar Tagen schon angeguckt und auch schon ne Lösung, hab aber erst intern einige Änderungen allgemein an Totems vorgenommen und hab die nächste Woche keine Zeit mich damit zu beschäftigen.

Der Commit den du rausgesucht hast ist von 2012, heißt die Änderung hatten wir schon, allerdings existiert der Code in der Form nicht mehr, sowohl auf Trinity als auch auf RG. Aber allgemein kann man sagen dass alle commits älter als 2015 useless sind^^
Trotzdem danke für den Versuch.

Aktionen #7

Von Connex vor fast 5 Jahren aktualisiert

  • Status wurde von In Bearbeitung zu Testbereit geändert
  • Zugewiesen an Connex wurde gelöscht
Aktionen #8

Von Jeora vor fast 5 Jahren aktualisiert

  • Status wurde von Testbereit zu Ready geändert
  • Update Text Deutsch aktualisiert (Vergleich)
Aktionen #9

Von Serverupdate vor fast 5 Jahren aktualisiert

  • Status wurde von Ready zu Pending geändert
Aktionen #10

Von Connex vor fast 5 Jahren aktualisiert

  • Update Text Deutsch aktualisiert (Vergleich)
Aktionen #11

Von Connex vor fast 5 Jahren aktualisiert

  • Tags wurde von Shaman, Spell zu Shaman, Spell, Update 2020-01-29 geändert
  • Status wurde von Pending zu Live geändert
Aktionen

Auch abrufbar als: Atom PDF