Skip to content

Conversation

@YuanRisheng
Copy link
Collaborator

Reverts #5064

Copilot AI review requested due to automatic review settings November 28, 2025 04:00
@paddle-bot
Copy link

paddle-bot bot commented Nov 28, 2025

Thanks for your contribution!

Copilot finished reviewing on behalf of YuanRisheng November 28, 2025 04:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

这是一个回退 PR,目的是撤销之前添加的单元测试(PR #5064)。该 PR 完全删除了 test_ep.py 测试文件。

关键变更

  • 删除了 tests/model_executor/test_ep.py 文件(645 行代码)
  • 移除了 DeepEP buffer 管理、MoE 选择逻辑等功能的测试覆盖
Comments suppressed due to low confidence (1)

tests/model_executor/test_ep.py:1

  • Pull Request 标题和描述问题

标题格式问题
当前标题为 "Revert \"[CI] 【Hackathon 9th Sprint No.18】NO.18 功能模块单测补充 -part\"\" 不符合规范格式。

建议修改为:[Revert] Revert Hackathon 9th Sprint No.18 unit test additions

描述信息不足
PR 描述仅包含 "Reverts #5064",缺少关键信息:

  1. 为什么要回退这个 PR?是因为测试失败、引入了 bug,还是其他原因?
  2. 这个回退解决了什么问题?
  3. 是否有计划重新引入这些测试?

建议补充完整的回退原因说明,例如:

## 回退原因
[请说明为什么需要回退 #5064]

## 影响范围
- 移除了 `test_ep.py` 中的所有单元测试(645行)
- 影响的测试覆盖:DeepEP buffer 管理、MoE 选择逻辑等

## 后续计划
[是否有计划修复并重新提交这些测试?]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants