执行摘要
- 一句话：删除 override_attention_dtype 遗留配置参数
- 推荐动作：值得快速浏览，作为 dead code 清理的标准范例：先确认下游使用已移除，再同步删除配置字段、CLI 参数与内部传递路径。无需精读，逻辑直白。
功能与动机
PR body 明确说明：override_attention_dtype is only used for V0 and has been removd from https://github.com/vllm-project/vllm/pull/25351/ long time ago. It's safe to remove it now. 即该配置项已无任何使用方，继续保留只会增加维护成本。
实现拆解
- 在 vllm/config/model.py 中移除 ModelConfig 的 override_attention_dtype 字段定义及 docstring，并从 compute_hash 的 ignored_factors 集合中删除该键（否则会引用不存在的属性）；同时删除 post_init 中针对非 ROCm 平台设置该配置时的 warnings.warn 告警块。由于告警块是文件中 warnings 的唯一使用处，import warnings 也随之删除。
- 在 vllm/engine/arg_utils.py 中删除 EngineArgs 类上的 override_attention_dtype 字段声明、add_cli_args 中 --override-attention-dtype 的 argparse 注册，以及 create_model_config 向 ModelConfig 构造传递该参数的代码。
- 未新增测试或文档，因为该配置已无实际行为分支，删除后不影响现有测试。分支在合并前经历了 6 次 main 分支合并以解决冲突，最终净变更为 -15 行。
关键文件：
vllm/config/model.py（模块 模型配置；类别 source；类型 data-contract；符号 ModelConfig.override_attention_dtype, ModelConfig.compute_hash, ModelConfig.post_init）: 核心配置变更：删除 ModelConfig.override_attention_dtype 字段、compute_hash 中的键及 post_init 的 ROCm 告警逻辑，同时移除不再需要的 warnings 导入。vllm/engine/arg_utils.py（模块 引擎参数；类别 source；类型 core-logic；符号 EngineArgs.override_attention_dtype, EngineArgs.add_cli_args, EngineArgs.create_model_config）: CLI 入口同步清理：删除 EngineArgs 字段、--override-attention-dtype argparse 注册及 create_model_config 中的参数传递。
关键符号：ModelConfig.override_attention_dtype, ModelConfig.compute_hash, ModelConfig.post_init, EngineArgs.override_attention_dtype, EngineArgs.add_cli_args, EngineArgs.create_model_config
评论区精华
该 PR 无实质性技术讨论，review 均为流程性内容：
mergify[bot]: “the pre-commit checks have failed. Please run ... pre-commit run --all-files” —— 触发了一次格式修复。
mergify[bot]: “This pull request has merge conflicts that must be resolved before it can be merged.” —— 通过反复合并 main 解决。
yewentao256: “you might need to enable CI by yourself /ci run” —— 作者在 fork 分支上无法由维护者代跑 CI，需自行触发。
最终 yewentao256 批准：LGTM, thanks for the work!
- pre-commit 失败与修复 (other): 作者后续提交修复了格式问题，pre-commit 通过。
- merge conflict 处理 (other): 通过 6 次 merge main 解决冲突。
- fork 分支 CI 触发 (question): 作者执行
/ci run 后 CI 正常触发，最后通过。
- PR 批准 (other): PR 被批准合并。
风险与影响
- 风险：主要风险是 CLI 兼容性：任何仍使用 --override-attention-dtype 的启动脚本或上层工具会收到 unrecognized arguments 报错。但由于该参数自 #25351 后已对 V0 无实际作用，且 V1 从未支持，实际受影响面极小。compute_hash 中移除该键不会改变哈希结果，因为对应字段已不存在，get_hash_factors 不会收集它。无回归、性能或安全风险。
- 影响：对用户：需要清理仍传递 --override-attention-dtype 的脚本，否则启动失败；对系统：无运行时影响，配置结构略微精简；对团队：减少一项需要长期维护的冗余配置，是 V0 遗留清理工作的一部分。
- 风险标记：CLI 参数移除, 配置契约变更, 无测试覆盖
关联脉络
- PR #25351 Remove override_attention_dtype usage in V0: PR body 明确说明 V0 侧对 override_attention_dtype 的使用已在该 PR 中移除，是本次清理的前置依赖。
参与讨论