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

Warlocks Infernal doing way less damage then intended #1299

Closed
Steadydnt opened this issue Jul 8, 2024 · 10 comments
Closed

Warlocks Infernal doing way less damage then intended #1299

Steadydnt opened this issue Jul 8, 2024 · 10 comments
Assignees

Comments

@Steadydnt
Copy link

Description:
Yo i noticed that the 10 minute Cooldown for Warlocks "Summon Infenal" doing way too less damage with their melee attacks. I just tested and melee attacks are average doing 700 damage a hit. While on Classic Wotlk and also all other Wotlk private servers its about 2000damage a melee attack. This bug makes the 10minute Cooldown almost not viable since you still loose a 1,2 sec casts for a Pet that will do basically 10k damage then your normal Pet.

How to reproduce:

How it should work:
Melee attack damage done by Infenal should be buffed.

Database links:

@disruption01
Copy link

I can confirm this issue.

@Inukk
Copy link

Inukk commented Jul 9, 2024

can confirm, both the damage and attack speed are bugged

@UltraNix
Copy link
Collaborator

UltraNix commented Jul 9, 2024

700 damage melee hit - against what?

@UltraNix UltraNix added the Dev has a question Further information is requested label Jul 9, 2024
@Inukk
Copy link

Inukk commented Jul 9, 2024

@UltraNix Here's the damage of an infernal in phase 1 of Classic at full t7 bis
image

And here's the damage of an infernal in Frostmourne, this log is from the current rank 5 affli on patchwerk
image

The infernal in frostmourne is doing roughly half the damage it should be doing compared to classic, this along with the bugged attack speed makes it so the overall dps is something like 35% of what it should be.

I also ran it through 2010's SimCraft for 3.3.5 release using an affli profile with gear similar to the one from the rank 5 log, and again the results show ~50% more damage on melee hits compared to what can currently be observed in frostmourne, it's also consistent with what can be seen in classic logs.
image

@UltraNix UltraNix added High Prio and removed Dev has a question Further information is requested labels Jul 10, 2024
@BalanCV
Copy link

BalanCV commented Jul 10, 2024

From my testing on PTR its simply that melees are not scaling with owner's SP.
Immolation aura is scaling fine its just the melees.

From wowsims:
Base stats:
stats.Strength: 331,
stats.Agility: 113,
stats.Stamina: 361,
stats.Intellect: 65,
stats.Spirit: 109,
stats.Mana: 0,
stats.MeleeCrit: 3.192 * core.CritRatingPerCritChance,

stats.AttackPower: ownerStats[stats.SpellPower] * 0.57,
stats.SpellPower: ownerStats[stats.SpellPower] * 0.15,

// infernal is classified as a warrior class, so we assume it gets the
// same agi crit coefficient
infernal.AddStatDependency(stats.Agility, stats.MeleeCrit, core.CritRatingPerCritChance*1/62.5)
infernal.AddStatDependency(stats.Strength, stats.AttackPower, 2)

MainHand: core.Weapon{
BaseDamageMin: 330,
BaseDamageMax: 494.9,
SwingSpeed: 2,
CritMultiplier: 2,
},

ALSO it mentions specifically that immolation is capped AOE. Not sure which type of cap so I assume the standard one.

@Manpower99
Copy link

From my testing on PTR its simply that melees are not scaling with owner's SP. Immolation aura is scaling fine its just the melees.

From wowsims: Base stats: stats.Strength: 331, stats.Agility: 113, stats.Stamina: 361, stats.Intellect: 65, stats.Spirit: 109, stats.Mana: 0, stats.MeleeCrit: 3.192 * core.CritRatingPerCritChance,

stats.AttackPower: ownerStats[stats.SpellPower] * 0.57, stats.SpellPower: ownerStats[stats.SpellPower] * 0.15,

// infernal is classified as a warrior class, so we assume it gets the // same agi crit coefficient infernal.AddStatDependency(stats.Agility, stats.MeleeCrit, core.CritRatingPerCritChance*1/62.5) infernal.AddStatDependency(stats.Strength, stats.AttackPower, 2)

MainHand: core.Weapon{ BaseDamageMin: 330, BaseDamageMax: 494.9, SwingSpeed: 2, CritMultiplier: 2, },

ALSO it mentions specifically that immolation is capped AOE. Not sure which type of cap so I assume the standard one.

Can confirm that this is very accurate wotlkclassic info with screenshots if required.

@UltraNix
Copy link
Collaborator

UltraNix commented Jul 18, 2024

The scaling is already okay.
I see the base stats are different.
But still - it;s not 50% difference.
@Inukk Can you send to me that profile?

@UltraNix UltraNix added the Dev has a question Further information is requested label Jul 18, 2024
@Steadydnt
Copy link
Author

I want to bring this up again mainly because I see you bring Legendaries & small buffs for a bunch of classes and forget about the only Cooldown Affliction has is still working wrong. It feels very bad to see this as Affliction Player.

Its still doing like half the damage then it should normally do. The Immolation Aura scales correct and also the Inferno Effect scales correct. Also the Attackspeed was fixed.
Infernal melee attacks are still calculated very badly. They are doing not even half the Damage then they should. The Calculation uptop sent by Manpower is probably correct i can just provide you screenshots again of how it should look like and how its looking here on Whitemane Private Wotlk.
This is Rank 1 affliction warlocks Infernal at KelThuzad in Wotlk Classic:
image

You can see its swinging autoattacks for like average 2k Physical damage. When it Crits on it they hit for about 4.4k damage.

Here im posting from my Kelthuzad Fight here on Warmane the value:
image

Its swinging 23x normal melee attacks for avg. 1073 damage. And when it critted 5x for avg. 2300 damage.

There is not much gear difference between my character on Whitemane and the character on classic. Its just exactly 50% less damage.

@Rowern96
Copy link

@UltraNix hey, since the base stats are clearly wrong and each strength difference translates in the attackpower being off by 2, isnt it possible to change the base stats and see how it effects the damage. its super annoying testing and gathering data on infernal while we already know one problem

@Manpower99
Copy link

I've already compiled good amount of resources to try and make an accurate bug report, it wont be long till i finalize my findings and report it and the rest is up to the devs to apply them or not. approximately a couple of days since im sadly very busy PTRing ulduar and raiding also. i will say however that the previous findings are not fully accurate either btw so no need to fully rely on them.

@UltraNix UltraNix added High Prio and removed Dev has a question Further information is requested labels Aug 22, 2024
@Piootrek Piootrek self-assigned this Aug 28, 2024
@Piootrek Piootrek closed this as completed Sep 2, 2024
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

9 participants