PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Duplicate skill gems in full DPS causes incorrect DPS calculations

Aperta
#4,682 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

bug
Lingua principale
Lua
Stelle
5.4k
Fork
2.4k
Merge medio
1g 12h
PR unite (30g)
26

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Importa il codice della build fornito da pobb.in, apri la scheda Skills e riproduci il problema abilitando due istanze di Toxic Rain in Full DPS e modificando una gemma di supporto. Confronta le variazioni del danno more/less visualizzate con il comportamento previsto mostrato negli screenshot; il lavoro è completato quando le gemme abilità duplicate non producono più moltiplicatori di DPS incoerenti o impossibilmente elevati.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
lua
Ambito
game-dev
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.