Skip to content

[CI]【Hackathon 10th Spring No.20】common_engine unit test [cf]#7698

Closed
ghost wants to merge 1 commit into
PaddlePaddle:developfrom
CloudForge-Solutions:task/h10-020-common-engine-test-v3
Closed

[CI]【Hackathon 10th Spring No.20】common_engine unit test [cf]#7698
ghost wants to merge 1 commit into
PaddlePaddle:developfrom
CloudForge-Solutions:task/h10-020-common-engine-test-v3

Conversation

@ghost

@ghost ghost commented May 2, 2026

Copy link
Copy Markdown

Motivation

Modifications

Usage or Command

Accuracy Tests

Checklist

  • I have submitted the CLA (only first PR)
  • My PR title follows the convention
  • My changes pass all tests

@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:14 — with GitHub Actions Inactive
@CLAassistant

CLAassistant commented May 2, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


boby-cloudforge seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:14 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:14 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:14 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:14 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@paddle-bot

paddle-bot Bot commented May 2, 2026

Copy link
Copy Markdown

Thanks for your contribution!

@paddle-bot paddle-bot Bot added the contributor External developers label May 2, 2026
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
@ghost
ghost temporarily deployed to Metax_ci May 2, 2026 17:15 — with GitHub Actions Inactive
PaddlePaddle-bot

This comment was marked as outdated.

@PaddlePaddle-bot

PaddlePaddle-bot commented May 2, 2026

Copy link
Copy Markdown

🤖 Paddle-CI-Agent | ci_status_monitor | 2026-05-03 21:54:57

CI报告基于以下代码生成(30分钟更新一次):


1 任务总览

所有 required 任务均通过。但有 7 个 Workflow 处于 action_required 状态,等待人工审批后方可执行,暂未产生实质 CI 结果。

总执行(rerun次数) 总任务 ✅ 通过 ❌ 失败 ⏳ 运行中 ⏸️ 等待中 跳过
1(0) 1 1 0 0 0 0

⚠️ 注意:以下 7 个 Workflow 处于 action_required 状态(等待审批后才会执行):ApprovalCodestyle-CheckCheck PR TemplateCI_HPUPR Build and TestCI_XPUILUVATAR-CI。这些 Workflow 需人工审批触发。

注意:action_required workflows 不计入上表的任务统计。


2 任务状态汇总

2.1 Required 任务 : 0/0 通过

未检测到必选任务(GitHub Branch Protection Rules 未返回 required 信息,或当前 API 权限不足)。

2.2 可选任务 — 1/1 通过

可选任务不阻塞合并,失败仅供参考。

状态 任务 耗时 日志 重跑
其余 1 个可选任务通过 - - -

3 失败详情(仅 required)

无 required 失败任务。


⚠️ 数据说明:获取 CI 数据时遇到网络异常(TLS 握手超时),部分 Workflow(run_id: 25280596892)的 Job 列表未能成功拉取,上述统计可能不完整。建议稍后刷新查看 CI 详情

@boby-cloudforge
boby-cloudforge force-pushed the task/h10-020-common-engine-test-v3 branch from fb18001 to 6fd8ce9 Compare May 3, 2026 13:34
@boby-cloudforge

Copy link
Copy Markdown

Motivation

No.20 功能模块 fastdeploy/engine/common_engine.py 单测补充

Modifications

add unittest tests/engine/test_common_engine.py

develop 分支:覆盖率83%,Miss行数199(82, 92-93, 98-99, 156-159, 174-175, 216, 234-235, 258, 263, 273->277, 291-294, 304-306, 311, 323->exit, 469->502, 601, 625->635, 630->632, 641->647, 651->653, 662->665, 706, 717, 822-823, 825-830, 832-833, 835-836, 855-856, 861, 864->869, 873, 877-879, 923->928, 937, 946-949, 958, 970->966, 1017->1002, 1018->1002, 1022, 1045-1047, 1053->1064, 1063, 1069-1070, 1074-1075, 1078, 1084->1099, 1104, 1130->1100, 1133-1134, 1140, 1155-1159, 1215->1218, 1224, 1234, 1239, 1267->1269, 1273->1275, 1299-1302, 1305-1308, 1310->1313, 1317->1319, 1324->1327, 1331, 1403, 1409-1410, 1419-1422, 1424-1426, 1428->1434, 1441, 1446-1457, 1475-1476, 1482-1491, 1512-1513, 1534->1545, 1595, 1630->1640, 1644-1645, 1671-1673, 1678->1691, 1706, 1710, 1738->1740, 1740->1748, 1742->1744, 1744->1748, 1767, 1768->1774, 1770->1774, 1791->1793, 1793->1796, 1794->1796, 1841-1842, 1873-1883, 1918, 1978, 1984, 2024, 2029->2028, 2056->2047, 2061, 2073-2079, 2081->2088, 2089-2096, 2110-2111, 2117->2121, 2124-2148, 2157-2161, 2178, 2185->2187, 2191, 2216->2230, 2230->2242, 2235->2242, 2243, 2300->2311, 2333->exit, 2382, 2384->2387, 2427, 2434-2435, 2436->2445, 2440, 2444, 2445->2448, 2501, 2549-2552, 2556->exit, 2565->2570, 2582, 2588->exit, 2603, 2633, 2641-2656, 2666->2668, 2672, 2679->2681, 2685, 2690-2691)

当前PR:覆盖率99%,Miss行数10

注:截图为本测试文件单独运行的 pytest --cov 输出(含 branch coverage),上方文字为 develop 已有测试与本 PR 合并后的 statement 覆盖率预估值,两者统计口径不同,以 CI 合并后实际值为准。

完成单测覆盖行数 199-10 = 189 → 四舍五入 200 → 预估贡献度 0.2⭐

Usage or Command

pytest tests/engine/test_common_engine.py

Accuracy Tests

no need

Checklist

  • Add at least a tag in the PR title.
    • Tag list: [[FDConfig],[APIServer],[Engine], [Scheduler], [PD Disaggregation], [Executor], [Graph Optimization], [Speculative Decoding], [RL], [Models], [Quantization], [Loader], [OP], [KVCache], [DataProcessor], [BugFix], [Docs], [CI], [Optimization], [Feature], [Benchmark], [Others], [XPU], [HPU], [GCU], [DCU], [Iluvatar], [Metax]]
    • You can add new tags based on the PR content, but the semantics must be clear.
  • Format your code, run pre-commit before commit.
  • Add unit tests. Please write the reason in this PR if no unit tests.
  • Provide accuracy results.
  • If the current PR is submitting to the release branch, make sure the PR has been submitted to the develop branch, then cherry-pick it to the release branch with the [Cherry-Pick] PR tag.

@cloudforge1

Copy link
Copy Markdown
Contributor

This PR is from a retired account and is superseded. Please refer to the active PRs for this task — see the latest open PRs from cloudforge1 or bob-cloudforge for the current version under review.

@luotao1 luotao1 closed this May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants