PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Sort gems by DPS not giving expected results

Abierto
#3,444 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

bug
Lenguaje dominante
Lua
Estrellas
5.4k
Forks
2.4k
Merge medio
1 d 12 h
PR fusionados (30 d)
26

Descripción

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

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Usa el código de build de Path of Building proporcionado y reproduce el problema añadiendo un séptimo enlace a Freezing Pulse con la ordenación por DPS activada. Sigue el comportamiento de ordenación al añadir gemas y verifica que las gemas se ordenen según el aumento de DPS mostrado, con Awakened Added Chaos, Awakened Fork y Empower en las posiciones esperadas.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
lua
Área
desktop
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.