Commit fa002af
M1: document tested-and-reverted DoT tick-order finding (Venom poison)
Investigated the Venom poison under-attribution (#11) against Spirit fixture
20260623_162050. Cadence is perfect (sim casts == real exactly), so the gap
is poison tick production: real 74 ticks vs sim 50.
Pinned the mechanic: 72 poison placements (Maneater A1 x40 + Venom A3 x2x16)
-> 74 real ticks ~= 1 tick/placement. Real poisons are detonation-consumed by
Venom A1 (tick once, then removed), NOT lasting 2 full turns.
Tested reordering the DoT damage loops before debuff_bar.tick() (deal on the
expiry turn): total -5.6% -> -1.2% but Venom flipped to +16.2% OVER (50 -> 94
ticks). That's a compensating-wrong swap (double-ticks long poisons), so
REVERTED. Left an explanatory comment at the tick site. The clean fix is
modeling poison detonation-consumption, validated against this fixture's
74-tick / 37.87M target — fixed together with the Geo deflect (downstream of
burns), not in isolation.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>1 parent 816f830 commit fa002af
1 file changed
Lines changed: 11 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1321 | 1321 | | |
1322 | 1322 | | |
1323 | 1323 | | |
| 1324 | + | |
| 1325 | + | |
| 1326 | + | |
| 1327 | + | |
| 1328 | + | |
| 1329 | + | |
| 1330 | + | |
| 1331 | + | |
| 1332 | + | |
| 1333 | + | |
| 1334 | + | |
1324 | 1335 | | |
1325 | 1336 | | |
1326 | 1337 | | |
| |||
0 commit comments