Prhub

#24145 [CI] Remove orphaned test/srt/ascend and test/srt/configs

原始 PR 作者 merrymercy 合并时间 2026-04-30 19:43 文件变更 9 提交数 1 评论 1 代码增减 +2 / -249

执行摘要

删除遗留测试目录并更新 NPU CI 路径

PR body 说明:Remove test/srt/ascend/ — the test file was only listed under not_in_ci in the legacy test/srt/run_suite.py and was never run in CI. NPU tests now live in test/registered/ascend/. Remove test/srt/configs/ — these YAML configs were left behind when experiment_runner.py and its workflow were deleted. Fix pr-test-npu.yml change-detection path from test/srt/ascend/to test/registered/ascend/ so that changes to NPU tests correctly trigger the CI workflow.

建议合并。这是一个积极的维护行为,可以快速通过。后续可检查是否还有其他残留的旧路径引用。

讨论亮点

无实质性讨论。仅有 gemini-code-assist[bot] 的自动审查评论,表示没有反馈。

实现拆解

  1. 删除 test/srt/ascend/test_embed_interpolate_unittest.py(104 行 NPU 测试文件)。
  2. 删除 test/srt/configs/ 下 5 个 YAML 配置文件:deepseek_v3.yaml, deepseek_v3_long_context.yaml, llama_405b.yaml, random_config.yaml, random_flashinfer_vs_triton_config.yaml, sharegpt_config.yaml。
  3. 修改 test/srt/run_suite.py,从 not_in_ci 列表中移除对已删除测试文件的引用。
  4. 修改 .github/workflows/pr-test-npu.yml,将路径 glob 从 test/srt/ascend/改为 test/registered/ascend/,以确保 NPU 测试变更能正确触发 CI 工作流。
文件 模块 状态 重要度
test/srt/ascend/test_embed_interpolate_unittest.py NPU 测试 removed 6.85
test/srt/configs/deepseek_v3.yaml 基准配置 removed 4.45
test/srt/run_suite.py 测试套件 modified 3.19
.github/workflows/pr-test-npu.yml CI 工作流 modified 2.38

关键符号

unpack TestEmbedInterpolate.setUpClass TestEmbedInterpolate.test_embed_interpolate

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

评论区精华

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

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

风险与影响

风险较低。被删除文件均已被确认无其他引用,且相关测试已迁移到新位置。CI 路径更新确保新测试能够触发。唯一轻微风险是如果存在未发现的引用,但 PR 作者已声明做了验证。

对最终用户无影响。对开发者而言,仓库减少混乱,构建时间略有缩短(删除无用文件)。CI 流程的 NPU 测试触发更准确。

删除遗留文件 CI 路径修复

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论