PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Duplicate skill gems in full DPS causes incorrect DPS calculations

Ouverte
#4,682 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

bug
Langage dominant
Lua
Étoiles
5.4k
Forks
2.4k
Merge moyen
1 j 12 h
PR mergées (30 j)
26

Description

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
What is the expected behaviour/value?

DPS Calculations should accurately show more/less changes to the skill.

What is the actual behaviour/value?

DPS appears to inconsistently either show the final calc values, or an impossibly large more damage multiplier.

How to reproduce the issue

If you wish to observe from nothing. The properties here appear to be slightly different. Ballista totem needs to be present as a link for a variant of the bug to appear.

  1. Add a bow to an empty build.
  2. Go to skills tab.
  3. Add 2 instances of Toxic Rain, one without links and the other with ballista totem. Enable "Include in FUll DPS" for both
  4. Start to add support gems to the first Toxic Rain.

In the PoB provided below, ballista totem does not appear to affect the outcome of the bug.

  1. Go to skills tab
  2. Select Toxic Rain (6)
  3. Attempt to change an enabled support to a new gem.

Steps to temporarily fix the bug from the PoB provided.

  1. Go to skills tab.
  2. Select TR Totem (5).
  3. Disable "Include in Full DPS."
  4. Switch to Toxic Rain (6).
  5. Attempt to change an enabled support gem to a new gem.
Character build code
https://pobb.in/hPql3eZ74-8F
Screenshots

Expected behavior
Current behavior

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Importez le code de build fourni depuis pobb.in, ouvrez l’onglet Skills et reproduisez le problème en activant deux instances de Toxic Rain dans Full DPS et en modifiant une gemme de support. Comparez les variations de dégâts more/less affichées avec le comportement attendu montré dans les captures d’écran ; le travail est terminé lorsque les gemmes de compétence dupliquées ne produisent plus de multiplicateurs de DPS incohérents ou impossiblement élevés.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
lua
Domaine
game-dev
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.