[Bugfix] Allow skipping MoE in NVFP4 (fix for MTP) (vllm-project#25987)

Signed-off-by: Benjamin Chislett <[email protected]>

Author: benchislett

vllm/model_executor/layers/fused_moe/layer.py

@@ -1194,6 +1194,8 @@ def __init__(
             if quant_config is None
             else quant_config.get_quant_method(self, prefix)
         )
+        if quant_method is None:
+            quant_method = UnquantizedFusedMoEMethod(moe)
 
         assert quant_method is not None
         assert isinstance(quant_method, FusedMoEMethodBase)

vllm/model_executor/layers/quantization/modelopt.py

@@ -884,8 +884,9 @@ def get_quant_method(
     ) -> Optional["QuantizeMethodBase"]:
         from vllm.attention.layer import Attention  # Avoid circular import
 
+        skip_layer = self.is_layer_excluded(prefix)
         if isinstance(layer, LinearBase):
-            if self.is_layer_excluded(prefix):
+            if skip_layer:
                 return UnquantizedLinearMethod()
             # Check if this is a vision model layer that should not be quantized
             if "vision_tower" in prefix or "vision_model" in prefix:
@@ -894,6 +895,8 @@ def get_quant_method(
         elif isinstance(layer, Attention):
             return ModelOptFp8KVCacheMethod(self)
         elif isinstance(layer, FusedMoE):
+            if skip_layer:
+                return None
             return ModelOptNvFp4FusedMoE(self, layer.moe_config, layer)
         return None
 

vllm/model_executor/models/deepseek_eagle.py

@@ -55,6 +55,7 @@ def __init__(
                 DeepseekV2DecoderLayer(
                     vllm_config,
                     prefix=maybe_prefix(prefix, f"layers.{i + start_layer_id}"),
+                    config=self.config,
                 )
                 for i in range(self.config.num_hidden_layers)
             ]

vllm/model_executor/models/deepseek_mtp.py

@@ -48,7 +48,8 @@ class DeepSeekMultiTokenPredictorLayer(nn.Module):
     def __init__(self, vllm_config: VllmConfig, prefix: str) -> None:
         super().__init__()
 
-        config = vllm_config.model_config.hf_config
+        config = vllm_config.speculative_config.draft_model_config.hf_config
+        self.config = config
         quant_config = vllm_config.quant_config
 
         self.enorm = RMSNorm(config.hidden_size, eps=config.rms_norm_eps)
@@ -66,11 +67,15 @@ def __init__(self, vllm_config: VllmConfig, prefix: str) -> None:
             )
         else:
             topk_indices_buffer = None
+
         self.shared_head = SharedHead(
             config=config, prefix=prefix, quant_config=quant_config
         )
         self.mtp_block = DeepseekV2DecoderLayer(
-            vllm_config, prefix, topk_indices_buffer
+            vllm_config,
+            prefix,
+            config=self.config,
+            topk_indices_buffer=topk_indices_buffer,
         )
 
     def forward(

vllm/model_executor/models/deepseek_v2.py

@@ -1055,11 +1055,13 @@ def __init__(
         self,
         vllm_config: VllmConfig,
         prefix: str,
+        config: Optional[DeepseekV2Config] = None,
         topk_indices_buffer: Optional[torch.Tensor] = None,
     ) -> None:
         super().__init__()
 
-        config = vllm_config.model_config.hf_config
+        if config is None:
+            config = vllm_config.model_config.hf_config
         model_config = vllm_config.model_config
         cache_config = vllm_config.cache_config
         quant_config = vllm_config.quant_config
@@ -1200,7 +1202,7 @@ def __init__(self, *, vllm_config: VllmConfig, prefix: str = ""):
         self.start_layer, self.end_layer, self.layers = make_layers(
             config.num_hidden_layers,
             lambda prefix: DeepseekV2DecoderLayer(
-                vllm_config, prefix, topk_indices_buffer
+                vllm_config, prefix, topk_indices_buffer=topk_indices_buffer
             ),
             prefix=f"{prefix}.layers",
         )