Prhub

#48220 [Misc] Remove dead code in ViT functionality test

原始 PR 作者 Isotr0py 合并时间 2026-07-10 19:17 文件变更 1 提交数 1 评论 0 代码增减 +0 / -438

test cleanup multi-modality

执行摘要

删除被跳过且冗余的 ViT 功能测试文件

PR body 指出该测试文件"always skipped"，并且已有 test_vit_cudagraph.py 覆盖了相同的 encoder CG 功能。删除死代码可以简化测试套件，避免误导开发者。

该 PR 是低风险的常规清理，值得合并。建议定期审计被跳过的测试，保持测试套件整洁。

讨论亮点

Review 中无实质性讨论。DarkLight1337 直接批准，claude[bot] 自动评论指出 fork 无法自动审查。

实现拆解

仅涉及一个文件删除：

  1. 删除测试文件：完全移除 tests/models/multimodal/generation/test_vit_backend_functionality.py。该文件包含以下内容：

    • 多个模型配置字典（如 dots_ocr、ernie45_vl、glm4_1v 等）
    • Prompt 构建辅助函数（build_dots_ocr_prompt、build_processor_prompt、build_ovis_prompt、build_qwen2_5_video_prompt）
    • 测试运行函数（run_llm_generate_test、run_llm_chat_test、run_video_test）
    • 主测试函数 test_vit_backend_functionality（被 @pytest.mark.skip 始终跳过）

  2. 无其他文件修改：没有导入调整或配置变更，因为该文件未被外部引用。

文件 模块 状态 重要度
tests/models/multimodal/generation/test_vit_backend_functionality.py 测试 removed 6.9

关键符号

test_vit_backend_functionality run_llm_generate_test run_llm_chat_test run_video_test build_dots_ocr_prompt build_processor_prompt build_ovis_prompt build_qwen2_5_video_prompt

分析完成后，这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论，后续有更多讨论时会体现在这里。

风险与影响

风险极低：

  • 被删文件始终被跳过，从未在 CI 中执行，移除不会导致测试覆盖损失。
  • 文件未被其他模块引用（通过 import 确认），不存在破坏依赖的风险。
  • 若未来需要类似测试，可参考 test_vit_cudagraph.py 或其他现有测试。

对用户无影响；对开发团队减少了约 440 行无用代码，降低了测试目录的噪声。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接，后续同步到相关引用后会出现在这里。

完整报告

Markdown PDF

参与讨论