已合并
embadding bag dispose core dump #1655
dong-yanrong创建于 2月6日
embadding bag dispose core dump #1655
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 dong-yanrong 的贡献)2月6日 添加了label:stat/needs-squash
2月6日 将FelixTang7,crystalhu,zhou-qilong,chenqi317,liubo75,tangweiwei2,wangyongguang,zhang-wu设为审查人
2月6日 将FelixTang7,crystalhu,zhou-qilong,chenqi317,liubo75,tangweiwei2,wangyongguang,zhang-wu设为审查人
CANN-robot
2月6日 评论:
2月6日 评论:
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 |
|---|---|---|
| index | ✅ chenqi317, tangweiwei2 (2/2) | ✅ tangweiwei2, chenqi317 (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)


2月6日 添加了label:cann-cla/yes
CANN-robot
2月6日 评论:
2月6日 评论:
🔵 source code change are detected, tasks labels is removed in this pull request!


此处折叠了54条消息 查看更多
CANN-robot
2月9日 评论:
2月9日 评论:
The following users do not have permission to comment /lgtm or /approve on any module in this PR:
songkai111


2月9日 添加了label:lgtm
CANN-robot
2月9日 评论:
2月9日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: tangweiwei2, chenqi317.
Reviewers who wrote a comment of /lgtm are: chenqi317, tangweiwei2.


2月9日 关闭了关联的issue
2月9日 合入了pull request
描述
Ascend950上embadding bag算子部分用例在测试geir通路以及精度的场景下,会出现core dump问题
原因:mean模板使用未初始化的内存,导致地址越界
修改方案:在mean模板使用特定内存前先初始化
关联的Issue
https://gitcode.com/cann/ops-nn/issues/901
测试
embeddingbag算子,本地白盒验证和网络用例验证测试geir 动态场景,静态场景,二进制通路均通过。
文档更新
类型标签