Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REC] Guardian druid removing Rage of the Sleeper from recommendation after using Incarnation #3933

Open
5 tasks done
aaronlilla opened this issue Oct 7, 2024 · 2 comments
Assignees
Labels
recommendation Ticket is a proposed change to action priorities. triage Issue needs initial triage to determine if action needs to be taken.

Comments

@aaronlilla
Copy link

Before You Begin

  • I confirm that I have downloaded the latest version of the addon.
  • I am not playing on a private server.
  • I checked for an existing, open ticket for this issue and was not able to find one.
  • I edited the title of this issue (above) so that it describes the issue I am reporting.
  • I am reporting an issue with the default priority included with the specialization (imported or edited priorities are not supported).

Spec

Druid - Guardian

Describe the Issue

Rage of the sleeper will be correctly recommended to be used after incarnation, pairing them together- until you actually use Incarnation and then Rage of the Sleeper will be totally removed from the recommendations until some later time. Sometimes it will constantly flash in the rotation for a second before disappearing again.

How to Reproduce

Enter game as Guardian Druid
Get into combat with 5 dummies pack
Note how it will recommend Thrash -> incarn -> rage of the sleeper etc..
Hit Incarnation, note how Rage of the sleeper is now removed from recommendations

Snapshot (Link)

https://pastebin.com/sttH7jqc

Raidbots Sim Report (Link)

No response

Additional Information

Before using Incarnation:
https://pastebin.com/sttH7jqc

After using Incarnation:
https://pastebin.com/Lz4E6h85

Contact Information

No response

@aaronlilla aaronlilla added recommendation Ticket is a proposed change to action priorities. triage Issue needs initial triage to determine if action needs to be taken. labels Oct 7, 2024
@syrifgit
Copy link
Collaborator

syrifgit commented Oct 8, 2024

The addon thinks the thrash spam is more important and doesn't make it down to the rage entry. Not idea if that's correct. but that's what is happening

@Hekili
Copy link
Owner

Hekili commented Oct 8, 2024

Syrif's description is correct; the reason the recommendations change suddenly is that initially, Thrash is on CD but Incarnation takes away its CD. That implies:

  1. The CD reset/elimination from Incarnation is probably not modeled correctly.
  2. As written, Rage of the Sleeper won't get used unless you have fully-stacked Thrash bleeds.

Possible solutions:

  1. Possible line_cd for Thrash so that it won't Thrash repeatedly, but that may backfire at other times (i.e., you don't have anything but Thrash to push anyway).
  2. Move Rage of the Sleeper up in priority above Thrash, but most of its conditions want you to have > 40 Rage, presumably to be able to Maul/Raze and make use of the Leech bonus. That means you could theoretically still Rage starve yourself enough to stay under 40 Rage and still keep Thrashing away, but it'd probably be harder.
  3. Could probably flag Rage of the Sleeper to use_off_gcd which would then allow it to weave in during a Thrash GCD, assuming its other requirements are still met.

Option 3 is probably the way that I'll go, but not 100% certain.

edit: Correction about why Thrash wasn't initially recommended.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
recommendation Ticket is a proposed change to action priorities. triage Issue needs initial triage to determine if action needs to be taken.
Projects
None yet
Development

No branches or pull requests

3 participants