PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Enabling "Include in Full DPS" breaks the gem comparison window for the main skill

Offen
#5,906 5 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

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
Check for support
  • I've checked that the behaviour is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What is the behaviour in-game?

N/A

What is the behaviour in Path of Building?

When turning on "Include in full DPS" the "new support gem window" comparison seems to compare the main skill to the extra skill enabled.

With Orb of Storms enabled:
image

With Orb of Storms disabled:
image

How to reproduce the issue
  • Create a new character
  • Add a Main skill and one set to "Include in full DPS"
  • With the Main skill selected add the Stun support (Since Stun support shouldn't alter anything it's easy to see the bug)
  • Notice the wild comparison numbers

Can also import the attached pob and enable the "Enabling this bugs out the Gem comparison window" socket group and see the bug happening with the main skill

Character build code
https://pobb.in/YXM10r2l6M09
Screenshots

Tested also on a new POB. Arc included in full dps:
image

Arc not included:
image

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

Beginne im Gem-Vergleichsfenster und reproduziere das Problem mit einem Hauptskill sowie einem weiteren Skill, für den "Include in full DPS" aktiviert ist, unter Verwendung von Stun support oder des verlinkten Build-Codes. Es werden keine Quelldateien oder Tests genannt; abgeschlossen ist die Aufgabe, wenn der Vergleich für den ausgewählten Hauptskill korrekt bleibt, unabhängig davon, ob der zusätzliche Skill in die vollständige DPS-Berechnung einbezogen wird oder nicht.

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.