Hekili / hekili

Hekili Priority Helper for DPS and Tanks (WoW Retail)
419 stars 221 forks source link

Vengeance DH Fel Desolation/Fel Devastation Hangs on Fel Devastation #3665

Closed thykk3r closed 1 month ago

thykk3r commented 2 months ago

Before You Begin

Describe the Issue

Hands on fel desolation when its actually on CD. Its either thinking it has multiple charges, is screwing up logic with fel devastation, but likely is missing a Cancel Meta Buff before it ( although sometimes that logic works)

How to Reproduce

Just play vengeance with my talents long enough and it will happen quickly. 896796796

Snapshot (Link)

https://pastebin.com/vttwUgV5

Raidbots Sim Report (Link)

No response

Additional Information

No response

Contact Information

No response

Redstoner875 commented 2 months ago

I have this same issue as well. Also in large pulls, it'll reccomend a LOT of soul cleave which is unoptimal afaik, and instead should build up souls for spirit bombs esp in meta. For example in the rookery, after you drop down after first boss, when I pull the entire area down to the second boss, it reccomends 2-3 spirit bombs, and then just says to spam soul cleaves.

dannym314 commented 2 months ago

Yes I have had this issue (perhaps fixed now?), and Redstoner's issue. Also it often is recommending my next 2 abilities to be Fracture even though I am capped on souls and fury. If there's some complex interplay that makes 2x Fracture my best option, I cannot find it, and it feels like it can't be the case.

Edit - I have captured the 2x Fracture issue:

I would make this a unique post, but there seems to be quite a few problems with VDH - such as not taking advantage of the new Demonsurge talent after you use Fel Devestation (ie. one spirit bomb, one soul cleave before leaving demon form to trigger the big AOE). There is also an order in which you should be using empowered abilities to take advantage of Demonsurge when in Meta, but it's a complex interplay of requirements.

I will try and capture the others when I have more time.

Hekili commented 2 months ago

Also in large pulls, it'll reccomend a LOT of soul cleave which is unoptimal afaik, and instead should build up souls for spirit bombs esp in meta.

Open a ticket for that with snapshots of it happening. I have to see why the addon is arriving at the recommendation to be able to analyze and resolve it.

The root cause of this ticket is likely that Fel Dev and Fel Des need to be distinguished from one another. I'll take a look.

Hekili commented 2 months ago

Fracture is recommended here because demonsurge_hardcast isn't applied, which is a virtual aura. I'll check to see that it is implemented/triggered correctly.

Hekili commented 2 months ago

1.0.13 will have major updates for Demonsurge builds.

dannym314 commented 2 months ago

FWIW all of the issues (including ones I hadn't mentioned yet) seem to have been fixed by this major update. Seems very smooth now.