Skip to content

Conversation

@WintersMontagne10335
Copy link
Contributor

@WintersMontagne10335 WintersMontagne10335 commented Nov 17, 2025

Motivation

NO.31 功能模块 fastdeploy/input/ernie4_5_processor.py 单测补充

Modifications

develop 分支:覆盖率58%,Miss行数107(53-58, 77, 92->96, 98-100, 106-107, 119, 130->137, 133->132, 136, 139, 142, 146, 155, 179-181, 187-188, 191->219, 193-205, 208->215, 214, 217, 220, 224, 229, 231, 252->254, 256-258, 262-266, 269, 283, 302->304, 318-322, 344-345, 348->363, 363->378, 364->366, 376->378, 380-383, 398, 408->410, 457->461, 458->457, 483-503, 509-517, 520-521, 526-561)
image

当前PR:覆盖率84%,Miss行数32
image

完成单测覆盖行数107-32=75

Usage or Command

no need

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.

@paddle-bot
Copy link

paddle-bot bot commented Nov 17, 2025

Thanks for your contribution!

@paddle-bot paddle-bot bot added the contributor External developers label Nov 17, 2025
@CSWYF3634076
Copy link
Collaborator

CSWYF3634076 commented Nov 18, 2025

191->219, 348->378 这两部分是否可以增加函数也完成覆盖呢

@WintersMontagne10335
Copy link
Contributor Author

191->219, 348->378 这两部分是否可以增加函数也完成覆盖呢

研发大大,其实已经基本覆盖了,您看:
image

@YuanRisheng YuanRisheng merged commit 4694ed2 into PaddlePaddle:develop Nov 19, 2025
15 of 16 checks passed
@CSWYF3634076
Copy link
Collaborator

0.1⭐️

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.

4 participants