herotc / hero-rotation

WoW Addon - Attempt to provide the useful and precise information to execute the best possible DPS rotation in every situation.
European Union Public License 1.2
166 stars 126 forks source link

[Bug]: With deathspeaker talented... recommendations get caught in a loop of constantly recommending sw:death #984

Closed GarrettCottrell closed 1 year ago

GarrettCottrell commented 1 year ago

What version of HeroRotation are you using?

10.1.5.11

What version of HeroLib are you using?

10.1.5.01

What version of HeroDBC are you using?

10.1.5.02

What is the issue?

image

using those talents, something weird happens with recommendations - shadow word death seems to be recoommended far more than I think is accurate... I can't say exactyly what's happening but there's soemthing going on i think with deathspeaker and shadow word death recommendations

Icon Behavior

None

Cilraaz commented 1 year ago

The only interaction between Deathspeaker and SW:Death is that SW:Death becomes a higher priority filler when the Deathspeaker buff is active. Enable debug output (/hr debug) and see if the SW:Death you think is occurring too often is "shadow_word_death filler 4".

GarrettCottrell commented 1 year ago

I will do that. To clarify, basically in my head it would make sense for sw:d to be recommended when the player has the deathspeaker buff but as soon as i press sw:d - that consumes the deathspeaker buff.

So, when I see shadow word death being recommended enough for me to press it 10+ times in a row something felt off becuase you dont get deathspeaker that often and on a full health enemy, you would use sw:d only when you have the buf

on that note, ill do what you just usggested and let you know what I get

GarrettCottrell commented 1 year ago

the output doesn't seem to register every shadow word death recommendation - particularly when im getting a bunch of them in a row. But mostly it was telling me shadow word death filler 3 (the first time i pressed it i got shadow word death filler 12)

GarrettCottrell commented 1 year ago

yeah its mostly saying filler 3

Cilraaz commented 1 year ago

There is no "filler 3".

GarrettCottrell commented 1 year ago

image

apparently there is? not sure

Cilraaz commented 1 year ago

You have an old version of the profile, then.

GarrettCottrell commented 1 year ago

Hm, okay - what would I need to update if that's the case?

Cilraaz commented 1 year ago

Update to 10.1.5.12 or grab a copy of Shadow.lua from the github.

GarrettCottrell commented 1 year ago

image on curseforge it doesn't seem to think I can upgrade past .11

Cilraaz commented 1 year ago

image

GarrettCottrell commented 1 year ago

Oh thanks - apologies for that, I hadn't gone into versions

GarrettCottrell commented 1 year ago

Yup -it's giving me filler 4 but when it goes into the loop that I previously identified as "weird behavior" it only outputs the first cast of sw:death and then when I follow the next couple sw:d recommendations it doesn't give me any output

GarrettCottrell commented 1 year ago

oh and then there are some filer 12's being recommended as well

Cilraaz commented 1 year ago

If it's the same suggestion, it won't spam the output. However, "filler 4" should only be suggested if the target is under 20% hp or if the Deathspeaker buff is up. Meanwhile, "filler 12" is SW:Death on the lowest hp target with Inescapable Torment talented and your fiend active.

GarrettCottrell commented 1 year ago

yeah so filler 4 on the first case makes sense but upon casting sw:d you consume the buff. so it seems super bizaree that it will recommend sw:d 3-4 casts in a row fairly often.

I'll get a better wa to track my deathspeaker buff and just try to confirm all of this

Cilraaz commented 1 year ago

I tested on a dummy with this talent setup and couldn't duplicate the issue. It would suggest SW:D, I'd use it, it would move to the next suggestion.

GarrettCottrell commented 1 year ago

yeah that happens often - im about to go into raid but maybe after i can try and capture a quick recording of the behavior happening