[CI]【Hackathon 10th Spring No.36】worker_process 单测补充#6736
Open
cloudforge1 wants to merge 8 commits intoPaddlePaddle:developfrom
Open
[CI]【Hackathon 10th Spring No.36】worker_process 单测补充#6736cloudforge1 wants to merge 8 commits intoPaddlePaddle:developfrom
cloudforge1 wants to merge 8 commits intoPaddlePaddle:developfrom
Conversation
|
Thanks for your contribution! |
cloudforge1
force-pushed
the
task/h10-036-worker-process-test
branch
from
3162a1f to
a39c57e
Compare
cloudforge1
force-pushed
the
task/h10-036-worker-process-test
branch
from
a39c57e to
9b0b166
Compare
cloudforge1
changed the title
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #6736 +/- ##
==========================================
Coverage ? 73.34%
==========================================
Files ? 399
Lines ? 55620
Branches ? 8766
==========================================
Hits ? 40792
Misses ? 11960
Partials ? 2868
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
cloudforge1
force-pushed
the
task/h10-036-worker-process-test
branch
from
e5e098f to
83e06af
Compare
cloudforge1
force-pushed
the
task/h10-036-worker-process-test
branch
from
168a166 to
906e312
Compare
…lat structure, 80% coverage - Flatten class-based tests to flat pytest functions (32→25 tests) - Add _run_eplb enabled path (token stats + weight update + value=None) - Add initialize_fd_config EP branch + quant_config branch - Add kv_cache cap at 40000, graph_optimize splitwise path - Add run_worker_proc with deterministic mode + iluvatar dispatch - Coverage: 69% → 80% (395/493 stmts)
Add test cases for: - EP config with int moe_num_experts (else branch) - Non-CUDA/XPU platform V1 scheduler fallback - v1_loader_support fallback to default - PaddleOCR architecture branch - num_hidden_layers=None ValueError
cloudforge1
force-pushed
the
task/h10-036-worker-process-test
branch
from
906e312 to
6022842
Compare
- Replace 33 MagicMock instances with types.SimpleNamespace, real numpy arrays, and real callables (lambda/async def) - Remove unittest.mock.MagicMock and AsyncMock imports - Keep only unittest.mock.patch for module-level substitution - All 21 tests pass, coverage unchanged at 80%
Contributor
Author
|
@luotao1 @CSWYF3634076 请review，谢谢！ |
…vcache lock, eplb elif — delta 122→152
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
No.36 功能模块 fastdeploy/worker/worker_process.py 单测补充
Modifications
add unittest tests/worker/test_worker_process.py
develop 分支：覆盖率61%，Miss行数169（85-87, 92-111, 134, 193-209, 300-314, 317-320, 332-385, 399-436, 452-453, 464-465, 486-488, 496->505, 500, 508-558, 568, 575, 582-585, 606-608, 620, 660, 683, 693-699, 713, 740-763, 1126-1127, 1132-1143, 1170, 1205-1206, 1234, 1259-1261, 1273-1277）
当前PR：覆盖率97%，Miss行数17
完成单测覆盖行数 169-17 = 152 → 四舍五入 200 → 预估贡献度 0.2⭐
Usage or Command
Accuracy Tests
no need
Checklist
[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]]pre-commitbefore commit.releasebranch, make sure the PR has been submitted to thedevelopbranch, then cherry-pick it to thereleasebranch with the[Cherry-Pick]PR tag.