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] Inconsistent Spellflash for spells that change on procs, such as Tempest, Ice Fury #4173

Open
5 tasks done
syrifgit opened this issue Nov 10, 2024 · 4 comments
Open
5 tasks done
Assignees
Labels
recommendation Ticket is a proposed change to action priorities. Suffer SpellFlash triage Issue needs initial triage to determine if action needs to be taken.

Comments

@syrifgit
Copy link
Collaborator

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

Shaman - Elemental

Describe the Issue

Spellflash doesn't always work, even with manually coded keybind. As far as I can tell, it seems to be when the spell proc happens while the spell is already recommended? Someone reported it in Discord and I was able to reproduce.

Screenshot of an instance where it worked

WoWScrnShot_110924_221602

Screenshot where it didn't work

WoWScrnShot_110924_222013

How to Reproduce

Just play any shaman build that uses tempest or Ice Fury, with Spellflash on.

Snapshot (Link)

https://pastebin.com/q5S6HUM2

Raidbots Sim Report (Link)

No response

Additional Information

No response

Contact Information

No response

@syrifgit syrifgit added recommendation Ticket is a proposed change to action priorities. triage Issue needs initial triage to determine if action needs to be taken. labels Nov 10, 2024
@Hekili
Copy link
Owner

Hekili commented Nov 10, 2024

Ok, your turn to drag SpellFlash's rotting corpse forward into TWW. :)

@Hekili Hekili added the Suffer SpellFlash label Nov 10, 2024
@PopsGG
Copy link

PopsGG commented Nov 16, 2024

Similar issue with spells changing on proc for blood DKs. I would love a fix if possible. Is SpellFlash abandoned?

@syrifgit
Copy link
Collaborator Author

Spellflash is an addon developed by someone else that is included with Hekili. See FAQ for more info on Spellflash.

@PopsGG
Copy link

PopsGG commented Nov 16, 2024

I know its a 3rd party library/addon, but its functionality works great with Hekili's core use case. I feel it is worth assisting with the fix or building the functionality directly into Hekili itself. If I knew how to code I would try to fix it myself. I ask humbly :)

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. Suffer SpellFlash triage Issue needs initial triage to determine if action needs to be taken.
Projects
None yet
Development

No branches or pull requests

3 participants