# PR #1992 完整报告

- 仓库：`THUDM/slime`
- 标题：cleanup
- 合并时间：2026-05-30 12:23
- 原文链接：http://prhub.com.cn/THUDM/slime/pull/1992

---

## 执行摘要

PR#1992 是一次代码清理，删除 MiniMax-M2.5 的两个转换脚本（HF↔Megatron），并将 CI 测试矩阵中的 `test_loss_cp_invariance.py` 替换为 `test_value_temperature.py`，同时在模板中补充该条目。整体变更幅度小，影响有限。

## 功能与动机

PR 标题“cleanup”表明这是维护性清理。删除的脚本曾是 PR#1929（MiniMax-M2.5 支持）的一部分，推测在合并后不再需要。CI 调整可能与 PR#1928 引入的新测试对齐，将旧的覆盖测试替换为新的值温度测试。

## 实现拆解

1. **删除转换脚本**：移除 `scripts/convert-minimax-m2-hf-to-megatron.sh` 和 `scripts/convert-minimax-m2-megatron-to-hf.sh`，共计 64 行。这两个脚本分别用于 HuggingFace 与 Megatron 格式之间的权重转换。
2. **更新 CI 配置文件**：在 `.github/workflows/pr-test.yml` 的零 GPU 测试矩阵中，将 `test_loss_cp_invariance.py` 替换为 `test_value_temperature.py`。
3. **同步模板**：在 `.github/workflows/pr-test.yml.j2` 模板中添加 `test_value_temperature.py` 条目，使模板生成的配置与手动维护的 YAML 一致。

### （无可用关键源码片段，变更主要为文件删除和配置列表替换。）

## 评论区精华

无 review 评论或讨论。

## 风险与影响

- **删除脚本**：若团队内部仍依赖这些脚本，可能造成工作中断。但鉴于 PR 被作者自行合并，应已验证无人使用。
- **测试覆盖变化**：`test_loss_cp_invariance.py` 被移除，若该测试仍有价值应确认其逻辑已迁移。新加入的 `test_value_temperature.py` 来自 PR#1928，相关测试能力得到补充。
- **影响范围**：仅影响 CI 测试集合和 MiniMax 模型转换脚本的可访问性，对核心逻辑无影响。

## 关联脉络

- **PR#1929**：新增 MiniMax-M2.5 支持，本次删除的脚本正是其中一部分。
- **PR#1928**：修复 critic value 温度问题，新增了 `test_value_temperature.py`，本次将其加入 CI 矩阵以增强测试覆盖。