Punishment Curse Should Be Automatically Calculated in DPS
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 55/100
Rechercherichtung
Beginne damit, den Punishment-Modifikator und das vorhandene Kontrollkästchen „Enemy is on Low Life“ in der DPS-Berechnung zu finden, und vergleiche diesen Pfad anschließend mit der automatischen Behandlung von Culling Strike. Als erledigt gilt die Aufgabe, wenn Punishment automatisch die Hälfte seines erhöhten Schadens bei niedrigem Leben beiträgt, bei aktiviertem Kontrollkästchen den vollständigen Wert verwendet und die Berechnung durch die relevanten Tests abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Check for duplicates
- I've checked for duplicate open and closed issues by using the search function of the issue tracker
What platform are you running Path of Building on?
Windows
Is your feature request related to a problem?
Punishment curse does not contribute to the final DPS unless the "Enemy is on Low Life" checkbox is checked. This behavior is inconsistent in PoB compared to effects like Culling Strike.
Describe the solution you'd like
I think the increased damage taken at half its value should be applied automatically.
For example, Culling Strike is calculated automatically in the final DPS. It does not require an "Enemy is in Culling Strike range" checkbox.
This should be especially easy now that Low Life is 50%. For example, if a player has level 1 Punishment that says:
"Cursed Enemies take 30% increased Damage while on Low Life"
It could automatically be calculated as:
"Nearby Enemies take 15% increased Damage"
Then, if the player checks the "Enemy is on Low Life" checkbox, it could use the original 30% value instead.
Please consider this change, and thanks a lot for all your hard work!
Describe alternatives you've considered
No response
Additional context
No response
- Vorherrschende Sprache
- Lua
- Sterne
- 5.5k
- Forks
- 2.4k
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus PathOfBuildingCommunity/PathOfBuilding
-
bug: accuracy
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
PathOfBuildingCommunity/PathOfBuilding#9911 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
PathOfBuildingCommunity/PathOfBuilding#7548 · 4 Kommentare · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug: calculation
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
PathOfBuildingCommunity/PathOfBuilding#10375 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 64/100
PathOfBuildingCommunity/PathOfBuilding#10362 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
PathOfBuildingCommunity/PathOfBuilding#10355 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in PathOfBuildingCommunity/PathOfBuilding
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
mogenson/PaperWM.spoon#178 ·
-
enhancement good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
-
severity: low
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
luainkernel/lunatik#1876 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 91/100
-
bug ready ticket
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Domktorymysli/vclu#5 ·