Skip to content

Commit 4864909

Browse files
[MM][Bugfix] Minor fix for VL model verification (#4384)
### What this PR does / why we need it? To fix ops test, where `model_config` has been set to `None` and doesn't has `hf_config` attribute, we have added a check for `model_config` to guarantee it is not `None_Type`. - vLLM main: vllm-project/vllm@2918c1b Signed-off-by: shen-shanshan <467638484@qq.com>
1 parent 463910e commit 4864909

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

vllm_ascend/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -810,7 +810,7 @@ def _is_contain_expert(config: Any):
810810
def is_vl_model(vllm_config: VllmConfig):
811811
"""Checks if the model is a VL model by config"""
812812
global _IS_VL_MODEL
813-
if _IS_VL_MODEL is None:
813+
if _IS_VL_MODEL is None and vllm_config.model_config:
814814
model_configs = vllm_config.model_config.hf_config.to_dict()
815815
_IS_VL_MODEL = "VL" in model_configs["architectures"][0]
816816
return _IS_VL_MODEL

0 commit comments

Comments
 (0)