PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Hit Rate with Max Unleash Seals is incorrect when Cast Rate is high enough

Abierto
#9,601 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

bug: calculation
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 open and closed issues by using the search function of the issue tracker
Check for support
  • I've checked that the calculation is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What platform are you running Path of Building on?

Windows

What is the value from the calculation in-game?

In-game when waiting for Max Unleash Seals, I believe Hit Rate should be calculated as the lower of the following two:

  1. 1 / Time to Gain Max Seals
  2. Cast Rate / (1 + Max Seals * 0.1)

So the value would be (with placeholder stats):

  • Time to Gain Max Seals = 1.5s
  • Cast Rate = 3/s
  • Max Seals = 8
  • 1 / Time to Gain Max Seals = 1 / 1.5 = 0.67 Hit Rate
  • Cast Rate / (1 + Max Seals * 0.1) = 3 / (1 + 8 * 0.1) = 1.67 Hit Rate
  • 0.67 is the lower of the two, so that should be the Hit Rate used
What is the value from the calculation in Path of Building?

In PoB, when config is set to Wait for Max Unleash Seals, PoB calculates Hit Rate as whichever is lower:

  1. 1 / Time to Gain Max Seals
  2. 1 / (Cast Rate * 1.1)

So it calculates the incorrect value (with placeholder stats):

  • Time to Gain Max Seals = 1.5s
  • Cast Rate = 3/s
  • Max Seals = 8
  • 1 / Time to Gain Max Seals = 1 / 1.5 = 0.67 Hit Rate
  • 1 / (Cast Rate * 1.1) = 1 / (3 * 1.1) = 0.30 Hit Rate
  • 0.30 is the lower of the two, so it chooses that (and so higher cast speed = lower hit rate and DPS currently)
How to reproduce the issue
  1. Select Arc as the main skill
  2. Add Unleash Support to Arc
  3. Tick on Do you wait for Max Unleash Seals?
  4. Increase cast speed until (Cast Rate * 1.1) > Time to Gain Max Seals (On new file, 100% increased cast speed is enough)
  5. See incorrect Hit Rate based on inverse of 110% of Case Rate
PoB for PoE1 build code
https://pobb.in/-OiTWyTB33fj
Screenshots

When Time to Gain Max Seals is greater than 110% of Cast Rate:
Image

When 110% of Cast Rate is greater than Time to Gain Max Seals:
Image

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

Reproduce el problema con el código de build de PoE1 enlazado usando Arc, Unleash Support y “Do you wait for Max Unleash Seals?”. Rastrea el cálculo de la tasa de impactos para esa configuración y verifica que use el menor valor entre la tasa de obtención de sellos y la expresión de la tasa de lanzamiento que incluye Max Seals; confirma el resultado con una alta velocidad de lanzamiento.

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

Evaluación

Stack tecnológico
lua
Área
game-dev
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.