已开启
添加dispatch_size参数校验 #907
Nurxat创建于 2月26日
添加dispatch_size参数校验 #907
已开启
Nurxat成员
2月26日
2月26日
- 修改前如果设置的dispatch_size大于batch size*n/dp会导致难以定位的通讯错误；
合并受阻
AtlasAccount
2月26日 评论：
2月26日 评论：
ascend-robot
2月26日 评论：
2月26日 评论：
Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
⚠️ This PR does not yet meet the following requirements:lgtm (requires ≥ 2 person(s) per module)、approve (requires ≥ 1 person(s) per module)
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-Ascend/MindSpeed-RL | ❌ (0/2)(You can also ask: zjchenn, weixin_44917616, Ginray1, xiecheng_1, dingzicha1997) | ❌ (0/1)(You can also ask: sz19991010, fengliangjun, zhoubeirong, elvinp, wucong25) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
2月26日 添加了label：ascend-cla/yes
ascend-robot
2月26日 评论：
2月26日 评论：
Nurxat
2月26日 评论：
2月26日 评论：
compile
AtlasAccount
2月26日 评论：
2月26日 评论：
已进入队列，请勿重复触发
2月26日 添加了label：ci-pipeline-running
2月26日 修改标题为 “添加dispatch_size参数校验”，原标题为“添加dispatch_size参数校验+规范文档”
2月26日 修改了pull request 的描述
AtlasAccount
2月26日 评论：
2月26日 评论：
正在运行中,预计 2026-02-26 17:03:41 运行结束,具体时间已实际情况为准,请勿重复compile
2月26日 删除了label：ci-pipeline-running
2月26日 添加了label：ci-pipeline-failed
ascend-robot
2月26日 评论：
2月26日 评论：
AtlasAccount
2月26日 评论：
2月26日 评论：
已进入队列，请勿重复触发
2月26日 删除了label：ci-pipeline-failed
2月26日 添加了label：ci-pipeline-running
AtlasAccount
2月26日 评论：
2月26日 评论：
正在运行中,预计 2026-02-26 21:46:12 运行结束,具体时间已实际情况为准,请勿重复compile
2月26日 删除了label：ci-pipeline-running
2月26日 添加了label：ci-pipeline-failed
ascend-robot
2月26日 评论：
2月26日 评论：
mindspeed_rl/config_cls/validate_config.py
@@ -372,0 +378,4 @@
378+ )
379+
380+ _validate_dispatch_size(rl_config.actor_rollout_dispatch_size, actor_config.global_batch_size, rl_config.n_samples_per_prompt, generate_config.data_parallel_size, "actor_rollout_dispatch_size")
381+ _validate_dispatch_size(rl_config.actor_logprob_dispatch_size, actor_config.global_batch_size, rl_config.n_samples_per_prompt, actor_data_parallel_size, "actor_logprob_dispatch_size")
这一行太长了，建议换行对齐参数，提高可读性。