PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

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

Offen
#948 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

bug
Vorherrschende Sprache
Lua
Sterne
5.4k
Forks
2.4k
Ø Merge
1 T. 12 Std.
Gemergte PRs (30 T.)
26

Beschreibung

  • 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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Reproduziere das Problem, indem du einen neuen Build erstellst, eine Minion-Fertigkeit mit Infernal Legion hinzufügst und in ihrem Dropdown-Menü auf einen anderen Support gehst. Verfolge die Einstiegspunkte für Vorschau und Gem-Auswahl, um herauszufinden, warum sich Main Skill ändert, und überprüfe anschließend, dass die Vorschau das Entfernen der aktuellen DPS-Quelle klar widerspiegelt und keine irreführenden Gesamtwerte mehr meldet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
lua
Bereich
desktop
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.