Guardian: Ursoc's Fury absorb is damage × (1 + mul) instead of damage × mul
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
Línea de trabajo
Empieza en engine/class_modules/sc_druid.cpp, en trigger_ursocs_fury_t, cerca de la línea enlazada, y compara el cálculo de absorción con el valor del talento descrito en el issue. Revisa los datos relevantes de talentos de Guardian y las pruebas o simulaciones existentes para confirmar el multiplicador esperado. El trabajo estará terminado cuando el escudo absorba alrededor del 38,6 % del daño de Thrash/Maul con los talentos indicados y coincida con el comportamiento en el juego.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Ursoc's Fury looks like it's giving way too big a shield. In trigger_ursocs_fury_t the shield is calculated as damage * (1 + mul), but the talent says it should be damage * mul:
With Nurturing Instinct and Natural Recovery taken, mul comes out to about 38.6%, so the shield should be ~39% of Thrash/Maul damage, but SimC gives ~139%.
We checked it against Guardian logs from M+ keys: in game Ursoc's Fury absorbs about 6% of max health per second, SimC shows about 23%, which lines up with the ~3.6x you'd expect from this.
This is on the midnight branch (d080afd, still the same on current midnight). Overriding the talent's effect value so (1 + mul) comes out to 0.386 brings it in line with the logs.
Thanks!
- Lenguaje dominante
- C++
- Estrellas
- 1.6k
- Forks
- 789
- Merge medio
- 8 h 51 min
- PR fusionados (30 d)
- 56
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de simulationcraft/simc
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
simulationcraft/simc#11978 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Arms slayer_st APL bugAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
simulationcraft/simc#11747 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
simulationcraft/simc#11935 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
simulationcraft/simc#11918 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Timewalking Item Level BugAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
simulationcraft/simc#11845 ·
Los mantenedores suelen responder en 1 día
Todos los issues de simulationcraft/simc
Issues similares
-
bug IO
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
MrNeRF/LichtFeld-Studio#3824 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
bug code-review testing tooling
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
aristocratos/btop#1945 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
boykopovar/AnyPS5#11291 ·
Los mantenedores suelen responder en 1 día