已合并
sort tiling int截断风险修复 #2970
黄晓彬创建于 5月27日
sort tiling int截断风险修复 #2970
已合并
黄晓彬创建于 5月27日
黄晓彬成员
5月27日

描述

sort tiling路由存在int64截断风险,需要增加防护代码

关联的Issue

https://gitcode.com/cann/ops-math/issues/1684

测试

1 280+ ST 用例精度通过,性能持平
2 二级冒烟通过

文档更新

不涉及

类型标签

  • Bug修复
  • 新特性
  • 性能优化
  • 文档更新
  • 其他,请描述:
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 黄晓彬 的贡献)
黄晓彬成员
5月27日 创建了 pull request,commit 44b98cfb
CANN-robot
CANN-robot成员
5月27日 评论:

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
math/sort 宋凯, wangrui_ (2/2) 宋凯, wangrui_ (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

ConanHuang, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robot成员
5月27日 添加了label:cann-cla/yes
CANN-robotCANN-robot成员
5月27日 将FelixTang7,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,songkai111,zhangyuxiang0119,llimwang,gubaocheng设为评审人
CANN-robotCANN-robot成员
5月27日 将FelixTang7,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,songkai111,zhangyuxiang0119,llimwang,gubaocheng设为审查人
此处折叠了46条消息 查看更多
CANN-robotCANN-robot成员
5月27日 添加了label:approved
RuiWang_成员
5月28日 评论:

/lgtm
/approve

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

Review Guide

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

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