已合并
AllReduce lastSlice calculate #749
chengyutao3创建于 3月5日
AllReduce lastSlice calculate #749
已合并
chengyutao3创建于 3月5日
已删除 :master合入到cann/asc-devkitmaster
chengyutao3
chengyutao3
3月5日

描述

修改了CcuPrepareForAllReduceM2M函数的核心计算逻辑。首先,将每个进程平均处理数据量tmpCount的计算由整数除法改为向上取整,确保所有数据都能被分配。其次,重构了最后一个分片大小lastSliceSize的计算逻辑,通过判断处理标准分片的进程数是否等于总进程数,来区分不同情况并精确计算非标准分片的大小,从而保证数据被完整、正确地处理,避免了数据丢失和越界访问。

测试

AllReduce红线case 100 条

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 chengyutao3 的贡献)
chengyutao3chengyutao3
3月5日 创建了 pull request,commit e7a169bb
CANN-robot
CANN-robot成员
3月5日 评论:

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

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/asc-devkit wqtshg_wt, yanqingshang (2/2) wqtshg_wt (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
3月5日 将zhanghao_0689,yanqingshang,wulinyu,zhuliangying,kong0808,dengjing_aoe,YeZZzzz1,bluesky901,wqtshg_wt,yangbinhua,gao_dafa,chenyiyuan,xun_zhuge,ookkkkoo1122,bianque,houyuyang,shi_nanxiang,wuyang_hw,ascendhjh,DragonBornHD84,jiangZD设为评审人
CANN-robotCANN-robot成员
3月5日 将zhanghao_0689,yanqingshang,wulinyu,zhuliangying,kong0808,dengjing_aoe,YeZZzzz1,bluesky901,wqtshg_wt,yangbinhua,gao_dafa,chenyiyuan,xun_zhuge,ookkkkoo1122,bianque,houyuyang,shi_nanxiang,wuyang_hw,ascendhjh,DragonBornHD84,jiangZD设为审查人
CANN-robotCANN-robot成员
3月5日 添加了label:cann-cla/yes
此处折叠了81条消息 查看更多
Yyanqingshang成员
3月5日 通过审查
王涛成员
3月5日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
3月5日 添加了label:lgtmapproved
CANN-robot
CANN-robot成员
3月5日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: wqtshg_wt.
Reviewers who wrote a comment of /lgtm are: wqtshg_wt, yanqingshang.

likedislike
CANN-robotCANN-robot成员
3月5日 合入了pull request