Punishment Curse Should Be Automatically Calculated in DPS
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 55/100
Piste de recherche
Commencez par localiser le modificateur de Punishment et la case à cocher existante « Enemy is on Low Life » dans le calcul des DPS, puis comparez ce cheminement avec la gestion automatique de Culling Strike. Le travail est terminé lorsque Punishment contribue automatiquement à hauteur de la moitié de ses dégâts augmentés contre les ennemis ayant peu de vie et utilise la valeur complète lorsque la case est activée, le calcul étant couvert par les tests pertinents.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- Lua
- Étoiles
- 5.5k
- Forks
- 2.4k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de PathOfBuildingCommunity/PathOfBuilding
-
bug: accuracy
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
PathOfBuildingCommunity/PathOfBuilding#9911 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
PathOfBuildingCommunity/PathOfBuilding#7548 · 4 commentaires · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
bug: calculation
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
PathOfBuildingCommunity/PathOfBuilding#10375 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 64/100
PathOfBuildingCommunity/PathOfBuilding#10362 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
PathOfBuildingCommunity/PathOfBuilding#10355 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de PathOfBuildingCommunity/PathOfBuilding
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Les mainteneurs répondent en général sous 3 jours
-
Campaign WC
Difficulté 2/5 1-3 heures Accessibilité débutants 60/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
public-transport/transitous#2569 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
core: a monitored method's raise rewrites every ? of its message, a resumed runtime's text includedPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverteseverity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
luainkernel/lunatik#1861 ·
Les mainteneurs répondent en général sous 1 jour