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

Death Knight Incorrect Ghoul Scaling #716

Closed
Cazera90 opened this issue Jun 16, 2024 · 2 comments · Fixed by #726
Closed

Death Knight Incorrect Ghoul Scaling #716

Cazera90 opened this issue Jun 16, 2024 · 2 comments · Fixed by #726

Comments

@Cazera90
Copy link

This one I'm not 100% sure on but I think I figured it out.

All of this was done WITHOUT the glyph of raise dead.

The sim is showing this for my ghoul's stats, 18102 attack power with only Horn of Winter:
inherit

Meanwhile ingame my ghoul has 13010 AP:
ghoulingame

In the sim it is saying the ghoul is inheriting 8585 strength which I believe is incorrect. When I use Horn of Winter I gain 720 str. My ghoul gains 1454 AP which seems almost appropriate for a 720 str gain. I don't know why its more, the pet sheet doesn't display str anymore.

The sim seems to show the ghoul having 476 base str. When my strength is added you get 6455 x 2 = 12910 AP which is very close to the 13010 on the pet sheet. I believe the ghoul gains 100% of the player's strength.

Also the Raise Dead for Blood/Frost have the same scaling as the base unglyphed unholy ghoul. I think on the current sim they both do have the same scaling but I'm not 100% sure. Frost is top and Unholy is bottom. Removed cloak as unholy so both have about the same strength:
raisedead
uhghoul

@Cazera90
Copy link
Author

After looking at the sim source code I found that the ghoul inherits 101% strength, has 476 base strength and -20 ap.

(((5979x1.01) + 476)x2)-20 = 13009.58 rounded up to 13010 is the exact AP my ghoul has.

I think this is causing the incorrect scaling, it is making the ghoul inherit 143.54% strength:
ghoul

@rosenrusinov
Copy link
Contributor

@Cazera90 Can you also provide your ghouls AP in the same set but with the glyph equipped?

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

Successfully merging a pull request may close this issue.

2 participants