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

Soulfrost spell power seems to not clear on weapon swap option #3344

Closed
OttoBoyWonder opened this issue Jul 19, 2023 · 2 comments
Closed

Soulfrost spell power seems to not clear on weapon swap option #3344

OttoBoyWonder opened this issue Jul 19, 2023 · 2 comments
Labels

Comments

@OttoBoyWonder
Copy link

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. On warlock sim, enable the weapon swap option
  2. Add a weapon to the weapon swap and add soulfrost enchant to it
  3. Run the sim
  4. Save to reference
  5. Review sim numbers
  6. Change the weapon swap enchant to mighty spell power
  7. Run the sim

Expected behavior
The difference between soulfrost and mighty spell power should not be so high in terms of sim. Perhaps the shadow spell power is 'sticking' after the weapon swap.

Sim Links and Screenshots
https://wowsims.github.io/wotlk/warlock/#eJztU0FoE0EU3f+zbjerJWMIOIa2xL0YhcIky0J7kN16McdoL6V46cEKEjR40niJVdFWUA8iNKhIvXhRGhE1VLFES+lBWwxEqpemsRixaohERVCc2SQ9SFGL9SJ+htn57/339/PnjxZQkSDDDohAD8QxCXAOYBRwHGAGoASw3aNCABnYEIU4XAPwSv4zygAo0Vjf4T0H1Sai+vOgecitKtLFVl2ZyOJEBjU3eTyHuusFP64jDzg339rwbr5EOsQ9EeYhmRLSz6qupDMo6GbytYL0oleXOSBxd4xrz7fUXQ8Zq6Du5sq0szhwn/NX2pb0HjI9h/RCyxIgk9NFUc6dIi8n7QCpotifFEUxlz8g/cKLmXWY/G0R+q5Qq1xvVoFBVO5ZcwhOAYxgFbVxlP7bH5mXrVSRUFUgSDEO+wxtCz/LlM9kkN9NB0ahdwO/9OufkJZ9unIvg3z5ZTJZRL9SCx2CzWHDZIyFmcE3wwyxsCkAwwiFzFB7uylIxszQPFD10kkXmV1AmlvAwEAOg9+eIjuawxLIZVibAimL0mB9Bjrt5yClUNJHXG4HSPbb62uUz96YGhY2ZXXWkBlrU79jeStcwfirJCaaumJ7+2IH9ge6qbz1Lw9WcNBejTTJqUe/yNP9M350unKsazXqWM6cRgd87+1hqPU+Y9X/VbYisGvZx3vEd2NHIXHc0p5djW/TJq1IISEQq6FrxD2sBnefzb61PopPdc5a3Hm37cSb19aP+f51W/nj7a0ryr/dq+/A471+

Link to sim above

@Tharre Tharre added the Warlock label Jul 19, 2023
@Tharre
Copy link
Contributor

Tharre commented Jul 19, 2023

Your assessment is correct, we use a bit of a hack of adding bonus SP to the spells themselves for school-specific SP since they are so rare in wrath, and that does not get reset when you swap weapons.

@jimmyt857
Copy link
Contributor

Reorganizing to #4082

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants