General's Cry Incorrect Behavior with Blade Flurry

Abierto
#5,935 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
lua
Área
game-dev

Línea de trabajo

Empieza en la ruta de Calcs para General's Cry y Blade Flurry, reproduce el Skill DPS inflado con la build de personaje enlazada y sigue cómo se aplican las etapas de canalización y la velocidad de ataque. Se considera terminado cuando el límite de canalización de 0,6 segundos y el límite de un ataque afectan al DPS, mientras el tooltip de velocidad de ataque permanece sin cambios.

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

Descripción

bug: behaviour
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
Check for support
  • I've checked that the behaviour is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What is the behaviour in-game?

Mirage Warriors will either:

  • Channel for up to 0.6sec -> release -> disappear (0.6sec channel time cap)
  • Channel up to 6 stacks -> release -> disappear (1 attack cap)
What is the behaviour in Path of Building?

It seems that Path of Building ignores the 0.6sec channel time cap and 1 max attack cap for Blade Flurry, instead it multiplies the average damage by the attack rate. This leads to a highly inflated DPS.

I suggest the following solution:

  • Cap the maximum number of stages providing a bonus to effective_max_stages = min(floor(attack_rate * general_cry_max_channel_time), base_max_stages).
  • Multiply the DPS by effective_max_stages/attack_rate to normalize it over 1 second, i.e. get the total damage done per 0.6sec of channeling, similar to non-channeled skills.

This way we can get a much more accurate read on the expected damage, without removing/modifying the tooltip about the attack rate, which is important for gearing, and the only remaining step to get the actual DPS is isolated to managing the warcry cooldown and spawning rate of the warriors.

How to reproduce the issue
  1. Add General's Cry, Blade Flury and equip a valid weapon
  2. Go to Calcs, select and highlight Skill DPS
Character build code
https://pastebin.com/L449UAZX
Screenshots

GeneralsCry

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

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.

Más de PathOfBuildingCommunity/PathOfBuilding

Todos los issues de PathOfBuildingCommunity/PathOfBuilding

Issues similares

Más issues de Lua

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.