PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Ramp Time in Side Bar
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Lua
- Estrellas
- 5.4k
- Forks
- 2.4k
- Merge medio
- 1 d 12 h
- PR fusionados (30 d)
- 26
Descripción
I'd like ramp time to be included whenever the calculation mode combines hit dps with poison/impale dps. It's important context for the final dps number that players can miss when talking about X bajillion viper strike dps.
If the box is getting too crowded then maybe replace the "total damage per poison" stat, which doesn't seem very relevant when dps is concerned.
ramp time for poison dps = poison duration
ramp time for impale dps = max number of impales / attacks per second
Originally posted by @Sectoidfodder in https://github.com/PathOfBuildingCommunity/PathOfBuilding/pull/158#issuecomment-594084168
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
No se nombra ningún archivo ni test. Empieza por localizar la visualización del modo de cálculo de la barra lateral y las estadísticas existentes de poison, impale y total-damage-per-poison; compara las fórmulas de ramp time solicitadas con los cálculos actuales. La tarea está terminada cuando los modos de DPS de hit combinado más poison/impale muestran la ramp time sin sobrecargar la barra lateral.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- lua
- Área
- desktop
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100