PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Duplicate skill gems in full DPS causes incorrect DPS calculations

Offen
#4,682 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

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

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

Importiere den bereitgestellten Build-Code von pobb.in, öffne den Tab Skills und reproduziere das Problem, indem du zwei Toxic Rain-Instanzen in Full DPS aktivierst und einen Support-Gem änderst. Vergleiche die angezeigten Änderungen am more/less-Schaden mit dem in den Screenshots gezeigten erwarteten Verhalten; die Aufgabe ist abgeschlossen, wenn doppelte Skill-Gems keine inkonsistenten oder unmöglich großen DPS-Multiplikatoren mehr erzeugen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
lua
Bereich
game-dev
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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