PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

DPS preview when swapping out lynchpin gem shows output for next default skill, sometimes misleadingly

Aperta
#948 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

  • What happened?

Messing with supports for a minion, I had their Infernal Legion DPS selected for Main Skill. When I then moved to the dropdown for picking a different support in the Infernal Legion slot, the preview tooltip claimed that changing to Elemental Focus would give me an unrealistic +1M DPS boost.

If I select the gem previewed, it changes my Main Skill to Minion Instability, with the resulting DPS matching the "+DPS inc. Poison" part of the strange preview. I'm not sure where the +1M Total DPS comes from.

  • What were you trying to do?

No clear use case here--you typically don't want to be swapping out the gem that gives you your "Main Skill" DPS anyway. It was an accidental encounter while exploring gem options. But in a less obviously-wrong case it could mislead someone about their DPS prospects!

  • What steps will reproduce the bug?
  1. Create a new build
  2. Add a minion skill and Infernal Legion support
  3. Go to the dropdown from Infernal Legion and hover over a different support. Note the possibly strange preview numbers
  4. Select the new gem. The Main Skill changes, with its numbers resembling the preview.
  • Does it reproduce every time? If not, why so?

Yes, although the specific numbers vary wildly depending on the rest of the build's setup.

  • Additional information:

image

https://pastebin.com/EnA8AuZk

Suggested fix would be to have any such preview say that it's removing all the DPS, making clear that you're effectively deleting that source of damage.

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

Riproduci il problema creando una nuova build, aggiungendo un’abilità di minion con Infernal Legion e passando il mouse su un altro support nel relativo menu a discesa. Segui i punti di ingresso dell’anteprima e della selezione delle gemme per capire perché cambia Main Skill, quindi verifica che l’anteprima rifletta chiaramente la rimozione dell’attuale fonte di DPS e non riporti più totali fuorvianti.

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

Valutazione

Stack tecnologico
lua
Ambito
desktop
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.