Fehler #35589
geschlossen
[Shaman][Spell] Fire totems and Fire Nova don't seem to scale with shaman crit
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
Von vaeterchenfrost vor etwa 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"
Von vaeterchenfrost vor etwa 5 Jahren aktualisiert
Logs that show high crit-percentage:
http://www.worldoflogs.com/reports/vweyx535vlctx49l/details/39/?enc=kills
http://www.worldoflogs.com/reports/vweyx535vlctx49l/details/75/?enc=kills
Von vaeterchenfrost vor fast 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 ;)
Von vaeterchenfrost vor mehr als 4 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...
Von Connex vor mehr als 4 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.
Von Connex vor mehr als 4 Jahren aktualisiert
- Status wurde von In Bearbeitung zu Testbereit geändert
- Zugewiesen an
Connexwurde gelöscht
Von Jeora vor etwa 4 Jahren aktualisiert
- Status wurde von Testbereit zu Ready geändert
- Update Text Deutsch aktualisiert (Vergleich)
Von Serverupdate vor etwa 4 Jahren aktualisiert
- Status wurde von Ready zu Pending geändert
Von Connex vor etwa 4 Jahren aktualisiert
- Tags wurde von Shaman, Spell zu Shaman, Spell, Update 2020-01-29 geändert
- Status wurde von Pending zu Live geändert