FastDeploy icon indicating copy to clipboard operation
FastDeploy copied to clipboard

[CI]【Hackathon 9th Sprint No.14】功能模块 fastdeploy/rl/rollout_model.py 单测补充

Open kesmeey opened this issue 1 week ago • 3 comments

Motivation

NO 14 fastdeploy/rl/rollout_model.py 单测补充

image dedevelop分支 ,覆盖率42,miss 行数 144行(81, 102, 123-131, 146, 155-210, 235, 255->261, 286-287, 294-295, 297-298, 305->307, 331, 340-365, 380, 389-439, 454, 462-487, 502, 510-535, 550, 559-615) image

当前pr,覆盖率91%,miss 行数 18

完成覆盖行数 144 - 18 =126

Modifications

NO 14 fastdeploy/rl/rollout_model.py 单测补充

Usage or Command

add unittest tests/rl/test_rollout_model.py

No need

Accuracy Tests

No need

Checklist

  • [x] 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.
  • [x] Format your code, run pre-commit before commit.
  • [x] Add unit tests. Please write the reason in this PR if no unit tests.
  • [x] Provide accuracy results.
  • [x] 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.

kesmeey avatar Dec 15 '25 06:12 kesmeey

Thanks for your contribution!

paddle-bot[bot] avatar Dec 15 '25 06:12 paddle-bot[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :warning: Please upload report for BASE (develop@80fb530). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #5552   +/-   ##
==========================================
  Coverage           ?   62.90%           
==========================================
  Files              ?      329           
  Lines              ?    41702           
  Branches           ?     6369           
==========================================
  Hits               ?    26234           
  Misses             ?    13485           
  Partials           ?     1983           
Flag Coverage Δ
GPU 62.90% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov-commenter avatar Dec 15 '25 08:12 codecov-commenter

image 当前失败的测试并不是当前pr的改动的文件

kesmeey avatar Dec 16 '25 02:12 kesmeey

0.1⭐️ @luotao1

CSWYF3634076 avatar Dec 18 '25 02:12 CSWYF3634076