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

NF Enh Shaman not correctly prioritizing earthen spike>FT #5707

Closed
kaptainm opened this issue Dec 29, 2020 · 0 comments
Closed

NF Enh Shaman not correctly prioritizing earthen spike>FT #5707

kaptainm opened this issue Dec 29, 2020 · 0 comments

Comments

@kaptainm
Copy link

Describe the bug

Line 53 in the APL causes fae transfusion to always be cast prior to earthen spike, resulting in what appears to be a ~0.5% DPS decrease if you just change that line to be earthen spike and add another line under it for just FT regardless. Not sure what the proper exact fix should be since it may vary b/w ST and AoE if earthen spike > FT should have highest prio or not.

Expected behavior

At least in ST, earthen spike should always come before FT (maybe only if they line up appropriately? not sure if the conduit that decreases the CD of FT which can put it out of sync would affect prio).

To Reproduce

Sim any NF enh shaman (has to be niya currently since dreamweaver doesn't have field of blossoms yet) and you'll see in the sample actions list that FT is always cast prior to earthen spike.

Additional information

  • SimulationCraft Version (required)
    Nightly and Weekly (most recent update to APL is more than a week old)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants