[Warrior] Unhinged's background Mortal Strike cannot proc Master of Warfare
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- Medio día
- Aptitud para principiantes
- 62/100
Línea de trabajo
Start in engine/class_modules/sc_warrior.cpp at the background mortal_strike_t constructor around line 3047 and the Master of Warfare roll around line 3164. Read how background is used by slam_base_t, execute_arms_t, and overpower_t, then determine whether the Unhinged auto-Mortal Strike needs a narrower exception rather than removing the guard broadly. Verify with an Arms warrior simulation that Master of Warfare can proc during Bladestorm without changing other background abilities.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
During Bladestorm channeling, the auto-cast Mortal Strike from Unhinged cannot proc Master of Warfare, because:
the Unhinged/background mortal_strike_t constructor sets background = true
the Master of Warfare roll is guarded by !background
Live combat logs show MoW procs during the Bladestorm channel at the same density as outside of it, so the !background guard appears to be too strict for this case.
Code references
engine/class_modules/sc_warrior.cpp (build 1210.01.73a372b):
cpp
// ~line 3047 — background ctor used for Unhinged
mortal_strike_t( util::string_view name, warrior_t* p )
: warrior_attack_t( name, p, p->talents.arms.mortal_strike ),
...
unhinged( true )
{
background = true;
cooldown->duration = 0_s;
proc_slayers_strike = true;
}
cpp
// ~line 3164 — Master of Warfare roll (inside mortal_strike_t)
if ( !background && p()->talents.arms.master_of_warfare_1.ok()
&& !p()->buff.master_of_warfare_proc->up()
&& p()->rng().roll( master_of_warfare_proc_chance * ++p()->master_of_warfare_attempts_since_last_proc ) )
{
p()->buff.master_of_warfare_proc->trigger();
p()->master_of_warfare_attempts_since_last_proc = 0;
}
The same !background guard exists for slam_base_t, execute_arms_t and overpower_t.
Game data
Master of Warfare (1269314) Proc Flags: Yellow Melee
Master of Warfare (1269391) Duration: 15 sec
Attributes: Can Proc From Procs (122)
Can Proc From Procs means proc-generated damage is allowed to trigger this proc, which is consistent with the Unhinged auto-MS being able to roll it.
- Lenguaje dominante
- C++
- Estrellas
- 1.6k
- Forks
- 789
- Merge medio
- 9 h 46 min
- PR fusionados (30 d)
- 55
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 78/100
simulationcraft/simc#11973 · 1 comentario ·
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
-
.NET ep:WebGPU platform:web
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
microsoft/onnxruntime#33755 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
grumpycoders/pcsx-redux#2530 ·
-
A-tools P-medium S-ready-to-start studio task
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Status: Needs confirmation Status: Needs triage Type: Code Quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Image.resolveAssetSource's TypeScript declaration drops the null it returns for an unknown assetAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
react/react-native#59648 ·
Los mantenedores suelen responder en 2 días