gamefreedomgit / Frostmourne

3 stars 1 forks source link

[DK] Rune of the Fallen Crusader #1244

Closed haephaistoss closed 2 weeks ago

haephaistoss commented 3 weeks ago

Description: PPM seems very high. It should be 1 PPM, now it's constantly applied...

How to reproduce: Equip the enchant and start hitting the dummy.

How it should work: It should equal roughly 2 PPM.

Database links: https://www.wowhead.com/wotlk/spell=53344/rune-of-the-fallen-crusader#comments

Manpower99 commented 3 weeks ago

it is very well known that it's 2 PPM same as original wotlk as classic https://web.archive.org/web/20100912010601/http://elitistjerks.com/f72/t94207-dk_mechanics_discussion/ since no one here wishes to be misled, It used to be 1PPM prior to Ulduar 3.1 In Ulduar however the strength was reduced to 15% and the PPM went from 1 to 2 Patch 3.1.0 (2009-04-14): Strength bonus reduced to 15%. Chance to proc doubled.

haephaistoss commented 3 weeks ago

Comment is from 3.3.2 though, let's assume you're correct, then:

True, I thought Ulduar wasn't live yet, my apologies. I'm only on the test server.

https://wowpedia.fandom.com/wiki/Rune_of_the_Fallen_Crusader?oldid=2290314 Can be seen here too.

Currently it's constantly on, even with simple auto attacks

Manpower99 commented 3 weeks ago

No need for assumptions i posted one elitist jerks findings and can post atleast 3 more. We are on 3.3.5a class balance regardless so it would be 2PPM from naxx to ICC. even on cataclysm it remained 2PPM. https://www.mmo-champion.com/threads/1345790-Rune-of-the-fallen-crusader-and-it-s-procs-uptime

Manpower99 commented 3 weeks ago

https://classic.warcraftlogs.com/reports/TD7ZCJ8zNxjav9rL#fight=18&type=auras&source=6 https://classic.warcraftlogs.com/reports/z17wmJNyfAaXq34G#fight=19&type=auras&source=14 every log on classic confirms it with over 80% uptime on frostdk and over 90% uptime on dual wield unholy.

WhitemaneIsa commented 3 weeks ago

@RadeghostWM opinion?

haephaistoss commented 3 weeks ago

Measured 18 applications in a 3 min testing window, with 100 % uptime.

That's too many even for 2 ppm currently.

Edited the main post to 2 ppm in accordance with recent findings.

RadeghostWM commented 3 weeks ago

We will check this. Despite being on 3.0 content we do use 3.3 balance, so it should be 2PPM. @Afgann

Manpower99 commented 3 weeks ago

Out of curiosity @Afgann if possible what PPM was it on prior?

Afgann commented 2 weeks ago

@Manpower99 it was 4PPM before.

Manpower99 commented 2 weeks ago

@Manpower99 it was 4PPM before.

Thank you!

Afgann commented 2 weeks ago

fixed