Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions vllm_ascend/quantization/quant_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,11 @@ def get_scaled_act_names(self) -> List[str]:
"language_model.lm_head.": "lm_head.",
"language_model.model.": "model.language_model.",
},
"qwen3_vl_text": {
"visual.": "model.visual.",
"language_model.lm_head.": "lm_head.",
"language_model.model.": "model.language_model.",
},
Comment on lines +213 to +217
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

While adding the prefix mapping for qwen3_vl_text is correct, a corresponding entry for this model type appears to be missing from packed_modules_model_mapping in this file (around line 312). Other dense Qwen VL models, such as qwen2_5_vl, have entries for packed modules like qkv_proj and gate_up_proj. If qwen3_vl_text also uses such packed modules, omitting it from packed_modules_model_mapping will likely cause a KeyError during quantization setup. Please consider adding an entry for qwen3_vl_text to packed_modules_model_mapping, similar to the one for qwen2_5_vl, to ensure correct handling of packed modules.

}

packed_modules_model_mapping = {
Expand Down
Loading