Projekt

Allgemein

Profil

Aktionen

Fehler #17730

geschlossen

[Druide] Wildwuchs wird nicht richtig berechnet mit T10

Von DerStefan vor etwa 8 Jahren hinzugefügt. Vor fast 4 Jahren aktualisiert.

Status:
Abgewiesen
Priorität:
Normal
Zugewiesen an:
-
Kategorie:
Zauber
Zielversion:
Beginn:
13.02.2016
Update Text Deutsch:
Tags:

Beschreibung

Der T2 2erboni ist auf RG nicht aktiv beim Baum...
da das ticke: (https://redmine.rising-gods.de/issues/14948) von den angaben her nicht ausreichend war wollte ich das noch mal aufnehmen...

Wildwuchs (https://db.rising-gods.de?spell=53251 )
t10 (http://www.wowhead.com/item=51300 ) / (https://db.rising-gods.de?item=51300)
t10 2er boni: (http://www.wowhead.com/spell=70658)

mit t10 3527 ZM :1054 , 996 , 937 , 880 , 864 , 851 , 836
58 59 57 16 13 15

Ohne t10 0 ZM : 376 , 339 , 302 , 267 , 321 , 193 , 156
37 37 35 36 38 37

Die unteren Zahlen sind immer die differenz zu dem nächstem heal.

und hier wie es sein sollte.

Wildwuchs Ticks verringern sich um ~38 pro Tick 30% davon sind 11.4. 38-11= 27

richtig wäre also mit einer zm von 3527:

1054 , 1027 , 1000 , 973 , 946 , 919 , 892

Bzgl Quellen: Tooltip + das aus Buffed + http://de.wowhead.com/spell=70658/item-druid-t10-restoration-2p-bonus#english-comments

Von Mollestus im alten ticket geschreiben:
"Aus den Kommentaren vom Offi:
Bsp:
Ohne T10 6 * (923 + 885 + 848 + 811 + 773 + 737 + 699) = 34044 Summe für 1 Ziel=5674 Delta Ticks: 38,37,37,38,36,38
Mit T10 6 * (923 + 897 + 871 + 845 + 818 + 792 + 766) = 35472 Summe für 1 Ziel=5912 Delta Ticks: 26,26,26,27,26,26"

Sprich auf dem Offi sind die Leute von der Differenz*0,7 ausgegangen, wie der Buffed User vorrechnete und wie es im Ticket hier beschrieben steht.
ich hoffe das die links nun reichen das es bearbeitet werden kann..
sollte dem nicht so sein reiche ich gerne weitere daten nach

Grüße Dabaum


Zugehörige Tickets

Beziehung mit RG Live WotLK 3.3.5a - Fehler #38510: Wildwuchs BerechnungBestätigt21.07.2020Aktionen
Aktionen #1

Von Rushor vor etwa 8 Jahren aktualisiert

  • Thema wurde von [Druide] Wildwuchs wird nicht richtig berechnet mit T10 zu [Druide] Wildwuchs wird nicht richtig berechnet mit T10 geändert
  • Kategorie wurde auf Zauber gesetzt
  • Status wurde von Neu zu Bestätigt geändert
  • Zielversion wurde auf Klassen gesetzt
  • Übergeordnetes Ticket wurde auf #4055 gesetzt
Aktionen #2

Von Rushor vor etwa 8 Jahren aktualisiert

  • DB-Link wurde auf https://db.rising-gods.de/?spell=53251 gesetzt
Aktionen #3

Von DevSargon vor etwa 8 Jahren aktualisiert

  • Status wurde von Bestätigt zu In Bearbeitung geändert
Aktionen #4

Von Rushor vor etwa 8 Jahren aktualisiert

  • Zugewiesen an wurde auf DevSargon gesetzt
Aktionen #5

Von DevSargon vor etwa 8 Jahren aktualisiert

  • Status wurde von In Bearbeitung zu Testbereit geändert
  • Zugewiesen an DevSargon wurde gelöscht

Der T10 Bonus sollte nun korrekt berechnet werden. Zusätzlich wurde die Berechnungsart der Ticks geändert damit sie wohl eher dem Stand von 3.3.5 entspricht: The base healing is split among the ticks with the first tick getting (6%+1/7) of the tooltip heal, and each successive tick losing 2% of the tooltip heal. Link: http://wowwiki.wikia.com/wiki/Wild_Growth (Dort ist es auch nur eine Referenz, leider ist der Link tot) Damit entspricht es auch ziemlich genau den Werten die in wowhead von marrowawe erwähnt werden zum 3.3.5 Patch. Link: http://www.wowhead.com/spell=70658/item-druid-t10-restoration-2p-bonus#comments (Nur die ohne T10 Bonus beachten, er rechnet den Bonus sehr seltsam drauf) Sollte das nicht passen gebt bitte auch eine genaue Berechnungsformel für jeden Wildwuchs-Tick an, denn mit der alten kam man zum jetztigen Stand (seltsamerweise) auch nicht mehr auf die öfters erwähnten 36-38 oder 26-27 mit T10-Buff.

Aktionen #6

Von Laederlappen vor etwa 8 Jahren aktualisiert

  • Status wurde von Testbereit zu In Bearbeitung geändert
  • Zugewiesen an wurde auf DevSargon gesetzt

272 ZM bei beiden Tests Die Zahlen daneben sind jeweils die Differenzen.
Ohne T10 2er Bonus:
251
246 5
241 5
237 4
233 4
228 5
223 5

Mit T10 2er Bonus:
251
247 5
244 3
241 3
237 4
235 2
232 3

Das ganze mit etwas mehr ZM getestet für bessere Differenzen:

Ohne T10 2er Bonus:
738
724 14
710 14
697 13
684 13
670 14
656 14

Mit T10 2er Bonus:
738
729 9
719 10
709 10
699 10
691 8
681 10

Mit dem Bonus sollte die Diff immer: Differenz*0,7 betragen. Das passt noch nicht genau. Wenn mgl das noch anpassen ansonsten sieht das besser als vorher aus.

Aktionen #7

Von DevSargon vor etwa 8 Jahren aktualisiert

  • Status wurde von In Bearbeitung zu Testbereit geändert
  • Zugewiesen an DevSargon wurde gelöscht

Ich sehe grad das die Formel sowieso nicht der gewünschten ganz entspricht, die Differenzen sind zu klein. Die sollten nun wesentlich grösser sein. Der Gesamtheal bleibt dadurch aber der Gleiche (ohne T10, mit ist er etwas höher). Die Formel sollte nun ähnlich hohe Werte wie bei marrowawe ergeben siehe: http://www.wowhead.com/spell=70658/item-druid-t10-restoration-2p-bonus#comments Momentan ist das jedoch immer die Differenz * 0.7. Durch die Kombination von Prozenten und Ganzzahlen entstehen halt teilweise Rundungsfehler. Ich hab jetzt noch eine andere Rundungsart gewählt, die möglicherweise etwas besser ist. Viel mehr kann ich aber in dieser Hinsicht nicht tun.

Aktionen #8

Von Laederlappen vor etwa 8 Jahren aktualisiert

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

Hmmm:
Ohne T10:
337
303 34
270 33
237 33
204 33
171 33
137 34

Mit T10:
337
313 24
290 23
267 23
244 23
220 24
197 23

Hmm sieht besser aus und kann denke ich so auf live

Aktionen #9

Von Laederlappen vor etwa 8 Jahren aktualisiert

  • Status wurde von Ready zu In Bearbeitung geändert
  • Zugewiesen an wurde auf DevSargon gesetzt

Sry kann doch nicht live.

"Aus den Kommentaren vom Offi:
Bsp:
Ohne T10 6 * (923 + 885 + 848 + 811 + 773 + 737 + 699) = 34044 Summe für 1 Ziel=5674 Delta Ticks: 38,37,37,38,36,38
Mit T10 6 * (923 + 897 + 871 + 845 + 818 + 792 + 766) = 35472 Summe für 1 Ziel=5912 Delta Ticks: 26,26,26,27,26,26"

mit 4272 ZM ohne T10:
990
892 98
795 97
697 98
595 102
502 93
402 100

Als Summe 4873, laut DrDamage sollte sie 4884 insgesamt heilen.

mit 4272 ZM mit T10:

990
921 69
853 68
785 68
717 68
648 69
580 68

Als Summe: 5494 (Laut Drdamage sollte es hier ~ 5069 sein)
Die Differenzen sind wohl zu groß, wenn man sich so Offiwerte anguckt^^
Dadurch wird Wildwuchs ohne T10 zwar fast blizzlike (Trotz großer Differenzen) und mit T10 wohl zu stark( weil 0,7*große Differenzen).

Aktionen #10

Von DevSargon vor etwa 8 Jahren aktualisiert

Wie gesagt, in dem Fall brauche ich schlichtweg eine neue Formel; Vorher kann ich gar nichts machen. Alternativ können wir auch die vorherige Formel nehmen dann ist es immer 29, bzw mit t10 rund 20

Aktionen #11

Von Veylen vor etwa 8 Jahren aktualisiert

Das sind halt keine festen Werte, wenn ich das richtig sehe...

Verringerung von erstem zum letzten Tick ohne T10 ~25%
Verringerung von erstem zum letztem Tick mit T10 ~18%

25*0,7=17,5 -> round it to 18%

Dementsprechend ist die Veringerung von Tick zu Tick immer bei rund 4,166% (25/6) ohne T10 bzw 2,916% ((25*0,7)/6) mit T10.

Aktionen #12

Von DevSargon vor etwa 8 Jahren aktualisiert

Am besten einfach die Rohwerte aus DrDamage angeben. Die Formel muss halt folgendes erfüllen: Gesamthaft muss man auf 700% Tickheal (weil es 7 Ticks sind) bzw 100% Gesamtheal kommen. Wenn ich einfach den ersten Tick als 100% nehme und den letzten dann als 75% ist das nicht erfüllt, und der Gesamtheal von Wildwuchs wäre geringer (Was deinem Vorschlag am nächsten käme Veylen wäre bei 115% zu beginnen und dann immer 5% runter; das könnte man natürlich so umsetzen)

Aktionen #13

Von Laederlappen vor etwa 8 Jahren aktualisiert

http://forums.elitistjerks.com/applications/core/interface/file/attachment.php?id=26974
Da gibts noch eine WG-Calculation ausm elitistjerks forum.
Die Formel müsste ja dann in der Excel Tabelle drin sein?

"WG numbers are consistent with the first tick gaining (1/7 + 6%) of the base damge. Each successive tick loses 2% of the base damage. The log contains ticks 2,4,5,6,7."
A closed form equation that predicts each of Norfair's WG ticks (to within less than +/- 1):

Tick_heal = (1442*(1/7+1/49*(4-tick_number))+SP*1.2*.115)*(1+2%*GoN+1%*Genesis)*(1+2%*MSS)*(1+6%*ToL)

The 1.2 is from ER (5/5 ER is required to reach WG). The 0.115 matches the wowwiki number.

Fully talented it turns into .115*1.2*1.15*1.04*1.06 = .175 which matches Norfair's number.

https://web.archive.org/web/20090612022108/http://elitistjerks.com/f73/t40221-resto_pve_healing_discussion/p59

Was mich jedoch irritiert ist das was in wowhead zuletzt stand:

http://de.wowhead.com/spell=70658/item-druid-t10-restoration-2p-bonus#english-comments

"Von marrowawe (154 – 1) am 2010.08.11 (Patch 3.3.5)
My Wild Growth does 1026-924-823-722-622-519-418 or so

The reduce rate is 97-102, average 100.
30% less reduce would be 30% more.
30*7 = 210 more heal per WG.

From 1026-924-823-722-622-519-418 = 5054
To 1056-954-853-752-632-549-448 = 5054+210= 5264.

On paper is small, next to useless."

Dann wäre es so, wie es jetzt wäre passend. Wurde iwas verändert dabei?

Aktionen #14

Von DevSargon vor etwa 8 Jahren aktualisiert

Deine Entscheidung was du willst. Ich kann die erwähnte von elitistjerks verwenden oder die jetzige behalten, die der von marrowawe
aus wowhead entspicht.

Aktionen #15

Von Veylen vor etwa 8 Jahren aktualisiert

Der letzte Kommentar auf WoWhead ist ziemlicher Blödsinn, da er nur einmal die 30% einrechnet.
Er reduziert halt nicht jedes mal um 30, sondern erhöht am Anfang um 30 und lässt dan den reduce so, wie ohne T10 -> Schwachsinn und deckt sich nicht mit Logs.

The base healing is split among the ticks with the first tick getting (6%+1/7) of the tooltip heal (before talents), and each successive tick losing 2% of the tooltip heal.

Das gilt NUR ohne Zaubermacht. Er sagt zwar auch "ohne Talente", das deckt sich aber nicht mit seinen eigenen Zahlen... Die ZM wird, wenn ich das richtig sehe und ihn richtig verstehe, auf jeden Tick einzeln angerechnet.
Und ich glaube DAS ist bei uns der Fehler.

Bei uns wird am Anfang einmal der Gesamtheal mit SP und Talenten berechnet und dann die Ticks nach dem Quote aufgeteilt, was in extrem hohen Gaps resultiert.

Aktionen #19

Von DevSargon vor mehr als 7 Jahren aktualisiert

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

Von Daejiv vor mehr als 7 Jahren aktualisiert

  • Status wurde von Testbereit zu In Bearbeitung geändert
  • Zugewiesen an wurde auf DevSargon gesetzt

Old Hash: 4dc59c037660bb1f87ccc4da6c537248b5b70437, b2202324764f13824af5f38fb6cf51592045f7da

Verursacht beim Script Load wohl Rnd Memory Corruptions, beim Script Validate.

Program received signal SIGSEGV, Segmentation fault.
0x00000000011ebdbf in _SpellScript::EffectHook::GetAffectedEffectsMask(SpellInfo const*) ()
(gdb) bt
#0  0x00000000011ebdbf in _SpellScript::EffectHook::GetAffectedEffectsMask(SpellInfo const*) ()
#1  0x00000000011f5284 in AuraScript::_Validate(SpellInfo const*) ()
#2  0x0000000000e26118 in ObjectMgr::ValidateSpellScripts() ()
#3  0x00000000010a0b73 in World::SetInitialWorldSettings() ()
#4  0x0000000000b5cc37 in Master::Run() ()
#5  0x0000000000add52e in main ()

Konnte das mit Debug Build leider nicht reproduzieren. Schau nochmal drüber, ob iwas anders ist oder fehlt im Vergleich zu den anderen Script Hooks.

Aktionen #22

Von Laederlappen vor mehr als 7 Jahren aktualisiert

ach deswegen funzte der Fix nicht :D

Aktionen #23

Von DevSargon vor mehr als 7 Jahren aktualisiert

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

Von Daejiv vor mehr als 7 Jahren aktualisiert

Master Reset

Aktionen #26

Von Daejiv vor etwa 7 Jahren aktualisiert

Master Reset

Aktionen #27

Von Daejiv vor etwa 7 Jahren aktualisiert

Master Reset

Aktionen #28

Von Daschluz vor etwa 7 Jahren aktualisiert

  • Zugewiesen an wurde auf Veylen gesetzt
Aktionen #29

Von Daejiv vor etwa 7 Jahren aktualisiert

  • Zielversion wurde von Klassen zu Klasse - Druide geändert
Aktionen #31

Von Veylen vor etwa 7 Jahren aktualisiert

  • Status wurde von Testbereit zu In Bearbeitung geändert
Aktionen #32

Von Veylen vor etwa 7 Jahren aktualisiert

Genereller Test von Wildwuchs:

0Sp, Genesis+Geschenk der Natur, nicht in Baum:
Ist: 337, 304, 270, 236, 203, 169, 135 1654
Soll: 339, 305, 271, 237, 203, 169, 135 1659
geringe Diskrepanz

27zm durch Base-Spirit, Genesis+Geschenk der Natur, in Baum (+Master Shapeshifter):
Ist: 376, 339, 301, 263 ,226, 198, 152 1867
Soll: 377, 340, 304, 266, 229, 191, 154 1861
geringe Diskrepanz

2195zm, Genesis+Geschenk der Natur, nicht in Baum:
Ist: 686, 652, 619, 585, 551, 517, 484 → 4094
Soll: 686, 653, 619, 585, 552, 518, 484 4097
geringe Diskrepanz

2356zm, Genesis+Geschenk der Natur, in Baum (+Master Shapeshifter):
Ist: 784, 747, 710, 673, 635, 598, 560 4707
Soll: 786, 748, 710, 673, 636, 599, 562 4714
geringe Diskrepanz

Mit leichten Varianzen von +-3 pro Tick stimmt Wildwuchs also.

T2:

2356zm, Genesis+Geschenk der Natur, in Baum (+Master Shapeshifter):
Ohne: 784, 747, 710, 673, 635, 598, 560 4707
→ 37, 37, 37, 38, 37, 38
Mit: 784, 823, 862, 902, 941, 980, 1019 6311 || ??? Das sollte abnehmen
→ 39, 39, 40, 39, 39, 39 kein Effekt

Fix funktioniert nicht

Aktionen #34

Von Veylen vor etwa 7 Jahren aktualisiert

  • Zugewiesen an wurde von Veylen zu WotLK - Developer geändert
Aktionen #36

Von Yrito vor etwa 7 Jahren aktualisiert

  • Status wurde von In Bearbeitung zu Bestätigt geändert
  • Zugewiesen an WotLK - Developer wurde gelöscht
Aktionen #37

Von Left4Info vor fast 6 Jahren aktualisiert

  • Tags wurde auf Classes, Druid gesetzt
Aktionen #38

Von Sockentester vor fast 6 Jahren aktualisiert

  • Zugewiesen an wurde auf Veylen gesetzt

setz dich da bitte mal direkt mit nem Dev zusammen, außer dir hat jeder den Überblick verloren, fürchte ich

Aktionen #39

Von LordMagnum vor mehr als 5 Jahren aktualisiert

  • Status wurde von Bestätigt zu Rückmeldung offen geändert
  • Update Text Deutsch aktualisiert (Vergleich)
Aktionen #40

Von Jeora vor mehr als 4 Jahren aktualisiert

  • Übergeordnetes Ticket #4055 wurde gelöscht
Aktionen #41

Von Jeora vor fast 4 Jahren aktualisiert

  • Status wurde von Rückmeldung offen zu Neu geändert
  • Zugewiesen an Veylen wurde gelöscht
Aktionen #42

Von Mezon vor fast 4 Jahren aktualisiert

  • Status wurde von Neu zu Abgewiesen geändert

Ticket aufgrund von Unübersichtlichkeit überarbeitet und als Ticket #38510 ausgelagert

Aktionen #43

Von Mezon vor fast 4 Jahren aktualisiert

  • Beziehung mit Fehler #38510: Wildwuchs Berechnung wurde hinzugefügt
Aktionen

Auch abrufbar als: Atom PDF