PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Sort gems by DPS not giving expected results

Aperta
#3,444 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

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.

image

image

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

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

Usa il codice della build di Path of Building fornito e riproduci il problema aggiungendo un settimo link a Freezing Pulse con l'ordinamento per DPS abilitato. Traccia il comportamento dell'ordinamento durante l'aggiunta delle gemme e verifica che le gemme siano ordinate in base all'aumento di DPS visualizzato, con Awakened Added Chaos, Awakened Fork ed Empower nelle posizioni previste.

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.