[dev] Winds of fate should change DPS, not average hit

Check version

• I'm running the latest version of Path of Building and I've verified this by checking the changelog

Check for duplicates

• I've checked for duplicate issues by using the search function of the issue tracker

How is Path of Building expected to behave?

When you deal 0% damage with non-crits, your average hit damage should not go down. It causes issues with ailment estimations. Instead, we should affect DPS.

Technically speaking, on average, it is true that the average damage goes down. However, in practice, we use average damage for both average damage per hit AND for average damage per hit that does damage, which is relevant for things like ignite, bleed, shock, etc.

How does Path of Building behave?

0% damage from non-crits when you're at less than 100% critical strike chance lowers the damage of average hit, which breaks ailments and leads to misleading breakdowns for things like burst damage, even though the final value is correct there.

How to reproduce the issue

No response

Character build code

https://pobb.in/MCD6Rlt0HAig

Screenshots

No response

In der Ausgabe wird keine Quelldatei und kein Test angegeben. Lade den verlinkten Build-Code, verfolge die Berechnungen für DPS und durchschnittlichen Trefferschaden bei nicht-kritischem Schaden unter Winds of Fate und vergleiche die Schätzungen für Ailments und die Schadensaufschlüsselungen vor und nach der Änderung. Als erledigt gilt die Aufgabe, wenn nicht-kritischer Schaden den durchschnittlichen Trefferschaden nicht mehr verringert und der finale DPS weiterhin korrekt ist.