FastDeploy icon indicating copy to clipboard operation
FastDeploy copied to clipboard

[CI]【Hackathon 9th Sprint No.12】功能模块 fastdeploy/spec_decode/mtp.py 单测补充

Open kesmeey opened this issue 2 weeks ago • 2 comments

Motivation

NO 12 fastdeploy/spec_decode/mtp.py 单测补充

image

develop分支,覆盖率56 ,miss行数199(40-50, 101, 104, 108, 135->137, 158, 196-197, 204, 207-233, 247-252, 286, 339->exit, 475, 498-505, 514, 552-558, 564-586, 597-674, 716-742, 748-751, 821-833, 870-882, 922, 946-948, 973-974, 989->996, 998, 1000->849, 1010-1091, 1094-1100, 1106-1129, 1143, 1161-1181, 1191, 1195, 1213-1214, 1219-1226)

image

当前pr,覆盖率81,miss行数 79

完成单测覆盖行数199-79=120

Modifications

add unittest tests/spec_decode/test_mtp_proposer.py

Usage or Command

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 12 '25 08:12 kesmeey

Thanks for your contribution!

paddle-bot[bot] avatar Dec 12 '25 08: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@e927c65). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #5533   +/-   ##
==========================================
  Coverage           ?   61.31%           
==========================================
  Files              ?      329           
  Lines              ?    41158           
  Branches           ?     6274           
==========================================
  Hits               ?    25238           
  Misses             ?    14030           
  Partials           ?     1890           
Flag Coverage Δ
GPU 61.31% <ø> (?)

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 12 '25 10:12 codecov-commenter

@freeliuzc 麻烦再看下修改是否符合要求

CSWYF3634076 avatar Dec 17 '25 02:12 CSWYF3634076

0.1⭐️ @luotao1

CSWYF3634076 avatar Dec 17 '25 12:12 CSWYF3634076