执行摘要
- 一句话：修正 DeepSeek-V4 模型注册测试的平台守卫条件，修复非 CUDA 平台 CI 失败。
- 推荐动作：该 PR 值得精读，因为它虽然改动微小（仅 1 行），却解决了一个影响面广的 CI 故障。它清晰地展示了如何为“仅部分平台支持”的模型架构设置正确的测试守卫。对于维护 vLLM 的多平台测试覆盖和 CI 健康度具有参考价值。
功能与动机
根据 PR body 描述，PR #54566 为 DeepSeek-V4 添加了一个仅支持 NVIDIA CUDA 的视觉实现，并在非支持平台上提供存根。但其对应的注册表测试 test_registry_imports 中的平台守卫逻辑被错误地反转，导致跳过了对真实 CUDA 实现的验证，却允许其他平台（如 ROCm）去验证不兼容的存根，从而引发多个 CI 构建失败（例如 Build 86967）。本 PR 的目的就是修正这一守卫条件，确保测试在正确的平台上执行。
实现拆解
变更仅涉及一个测试文件，是针对控制流逻辑的直接修正：
- 入口：修改位于
tests/models/test_registry.py 的 test_registry_imports 函数内，针对 DeepseekV4ForConditionalGeneration 模型架构的平台判断逻辑。
- 核心逻辑调整：将原有的条件判断
if model_arch == "DeepseekV4ForConditionalGeneration" and ( current_platform.is_cuda() ): 修改为 if model_arch == "DeepseekV4ForConditionalGeneration" and not ( current_platform.is_cuda() ):。具体地，在 and 后添加了 not 关键字。
- 效果：修正后的逻辑确保了仅在非 CUDA 平台（如 ROCm、XPU）上才执行
pytest.skip("Deepseek V4 is only supported on CUDA")，从而跳过不支持此模型的测试。在 CUDA 平台上，守卫条件不满足，测试将继续执行并验证真实的 DeepSeek-V4 模型类注册和接口兼容性，这符合模型的实际支持情况。
关键文件：
tests/models/test_registry.py（模块 注册表测试；类别 test；类型 test-coverage）: 这是唯一被修改的文件，包含了修复模型注册测试平台守卫逻辑的核心变更。
关键符号：未识别
关键源码片段
tests/models/test_registry.py
这是唯一被修改的文件，包含了修复模型注册测试平台守卫逻辑的核心变更。
# 位于 tests/models/test_registry.py 的 test_registry_imports 函数中
# 修复：将原有的 `and (` 改为 `and not (`，确保在非 CUDA 平台跳过测试
if model_arch == "DeepseekV4ForConditionalGeneration" and not (
current_platform.is_cuda()
):
pytest.skip("Deepseek V4 is only supported on CUDA")
# 确保所有 vLLM 模型类可以被成功导入
model_cls = ModelRegistry._try_load_model_cls(model_arch)
assert model_cls is not None
# ... 后续检查模型类型（文本生成、多模态、池化等）
评论区精华
Review 讨论很少，表明变更简单明确。
风险与影响
关联脉络
- PR #54566 [New model][Multimodal] Add DeepSeek-V4-Flash-Vision-Exp support: 本 PR 修复的逻辑错误直接源自此 PR，它为 DeepSeek-V4 添加了仅 CUDA 支持的视觉实现并引入了反转的平台守卫。
- PR #54177 无法从提供的上下文中获取确切标题，但被关联为导致 CI 失败的 PR: PR body 中指出此 PR（以及 #54651, #54962）的 CI 构建因本 PR 所修复的问题而失败。
参与讨论