PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Sort gems by DPS not giving expected results
Dieses Issue hat noch niemand übernommen.
- 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?
Adding a gem will be sorted in descending order based on dps.
What is the actual behaviour/value?
Awakened Added Chaos is appearing before other gems that provide more dps. Other gems such as Awakened Fork and Empower are also appearing out of order.
How to reproduce the issue
Attempting to add a 7th link to freezing pulse in order to simulate Uul-Netol's Vow. With sort gems by dps checked, awakened added chaos (~15%) is appearing above gems that are clearly better, eg inc crit strikes at ~36%
Build code
https://pastebin.com/YvdbiaM2
Screenshots
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Verwende den bereitgestellten Path of Building-Build-Code und reproduziere das Problem, indem du Freezing Pulse einen siebten Link hinzufügst und die Sortierung nach DPS aktivierst. Verfolge das Sortierverhalten beim Hinzufügen von Gems und überprüfe, dass die Gems nach ihrer angezeigten DPS-Steigerung angeordnet sind, wobei Awakened Added Chaos, Awakened Fork und Empower an den erwarteten Positionen stehen.
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