Commit f68273d
authored
[BugFix] Add compatibility guard for unsupported CLI params on Ascend NPU (vllm-project#10602)
## What this PR does
Add silent reset handling in `_fix_incompatible_config` for four
upstream CLI parameters that are incompatible with Ascend NPU:
| Parameter | Config Section | Reset Value | Reason |
|-----------|---------------|-------------|--------|
| `--calculate-kv-scales` | Cache Config | `False` | Deprecated
upstream; not supported on Ascend |
| `--compilation-config.use_inductor_graph_partition` | Compilation
Config | `False` | Ascend forces `use_inductor=False`, making this
irrelevant |
## User-facing change
Users who pass `--calculate-kv-scales`, `--compilation-config
'{"use_inductor_graph_partition": true}'` on Ascend NPU will now see a
warning log and the parameter will be silently reset to its safe
default, rather than potentially causing errors or undefined behavior.
## How was this patch tested
- Code review of the `_fix_incompatible_config` method to verify the
pattern matches existing silent-reset entries.
- vLLM version: v0.22.1
- vLLM main:
vllm-project/vllm@967c5c3
---------
Signed-off-by: underfituu <hzhucong@163.com>1 parent f6a737c commit f68273d
1 file changed
Lines changed: 16 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1049 | 1049 | | |
1050 | 1050 | | |
1051 | 1051 | | |
| 1052 | + | |
| 1053 | + | |
| 1054 | + | |
| 1055 | + | |
| 1056 | + | |
| 1057 | + | |
| 1058 | + | |
1052 | 1059 | | |
1053 | 1060 | | |
1054 | 1061 | | |
| |||
1224 | 1231 | | |
1225 | 1232 | | |
1226 | 1233 | | |
| 1234 | + | |
| 1235 | + | |
| 1236 | + | |
| 1237 | + | |
| 1238 | + | |
| 1239 | + | |
| 1240 | + | |
| 1241 | + | |
| 1242 | + | |
1227 | 1243 | | |
1228 | 1244 | | |
1229 | 1245 | | |
| |||
0 commit comments