Repository navigation
Fix int4 MoE tuner config filename - #35260
Conversation
Signed-off-by: maithilijoshi20 <maithilij2003@gmail.com>
|
@JeremiahM37 This PR removes the extra int4-specific halving from both the tuner lookup and generated config filename, so the tuner now emits the runtime key (for example, E=256,N=256). Would you be able to try your reproduction and confirm it resolves the issue on your setup? |
|
Confirmed, this fixes it — thanks for picking it up. I couldn't re-run the full tune+serve (my ROCm image is gone), so I verified the filename logic directly against the model from the report (E=256, which is exactly the key the server was looking for. Good catch on the second copy in |
|
@BBuf Could you help review or route this small fused-MoE tuner config fix? The issue reporter validated the filename correction. |
Thanks for your fix. |
Fixes #35252
Summary
Testing
The config-only test uses a stub for sgl_kernel because the available binary wheel targets a different GPU architecture; no kernel execution is covered by this change.
CI States
Latest PR Test (Base): ❌ Run #32810867033
Latest PR Test (Extra): ❌ Run #32810866824
Latest PR Test (AMD ROCm 7.2): ❌ Run #32810867026