已合并
[bugfix] AddLora新增对于参数ySliceSize和yOffset的校验 #2973
qiansunchi159创建于 5月27日
[bugfix] AddLora新增对于参数ySliceSize和yOffset的校验 #2973
已合并
qiansunchi159创建于 5月27日
qiansunchi159
5月27日

描述

  1. 增加对于ySliceSize和yOffset和H3关系的校验
  2. 对于ySliceSize和yOffset,先用int64判断正负情况,再进行后面的校验

关联的Issue

关联issue #1579

测试

见门禁冒烟

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 qiansunchi159 的贡献)
Qqiansunchi159
5月27日 创建了 pull request,commit bc13e66a
Qqiansunchi159
5月27日 关联了issue:[Bug-Report|缺陷反馈]: AddLora 的 y_slice_size 默认值与 y_offset 写回边界校验不一致,存在默认参数失败和越界写回风险
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/add_lora ✅ 张浩, 查建青 (2/2) ✅ 查建青 (1/1)

💡 Tip:

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

CLA Signature Pass

qiansunchi159, 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_,zhajianqing123,rxtfeng,luowen203_gg123,loov1,zhangzijie,jia0liang,songkai111,zhangyuxiang0119,demoauguste,llimwang,gubaocheng设为评审人
此处折叠了63条消息 查看更多
CANN-robot
CANN-robot成员
5月28日 评论:

The following label is not ready.

lgtm: Please wait for reviewers to review the code.

likedislike
demoauguste成员
5月28日 评论:

/lgtm

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: 查建青.
Reviewers who wrote a comment of /lgtm are: 查建青, 张浩.

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