已合并
fix A2 bevpoolv3 bug with large channel #2075
fix A2 bevpoolv3 bug with large channel #2075
已合并
gitdzy创建于 6月3日
gitdzy
gitdzy成员
6月3日

What this PR does / why we need it?

bugfix: fix A2 bevpoolv3 bug with large channel.

Does this PR introduce any user-facing change?

no.

How was this patch tested?

tested on 200 generalized cases.

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 gitdzy 的贡献)
gitdzygitdzy成员
6月3日 创建了 pull request,commit bb31450b
ascend-robotascend-robot成员
6月3日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
6月3日 评论:

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](https://gitcode.com/Ascend/community/blob/master/MindSDK AI应用软件开发套件/sigs/DrivingSDK/sig-info.yaml)


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
kernels ✅ 华郁秀, zhang_qianqian46 (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

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

likedislike
gitdzygitdzy成员
6月3日 关联了issue:[Roadmap] DrivingSDK Roadmap 2026 Q2
gitdzy
gitdzy成员
6月3日 评论:

compile

likedislike
ascend-robotascend-robot成员
6月3日 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
6月3日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
6月3日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
6月3日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robot
ascend-robot成员
6月3日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
6月3日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
6月3日 添加了label:docs-ci-pipeline-success
ascend-robot
ascend-robot成员
6月3日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
6月3日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
6月3日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
6月3日 评论:
流水线 PR-pipeline_DrivingSDK#1286 [ commitID:11bf766a ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
编译构建 Build_x86 ✅ >>>
Build_ARM ✅ >>>
恶意代码检查 Antipoison ✅ >>>
编码安全与规范检查 CodeCheck ✅ >>>
CodeCheck_pre-commit ✅ >>>
开源片段检查 SCA ✅ >>>
开发者测试 UT_ARM ✅ >>>
流水线 PR-pipeline_DrivingSDK ✅ >>>
此流水线已支持下列评论快捷指令,仅PR创建者评论有效
  • compile : 运行流水线
likedislike
zhang_qianqian46成员6月3日进行代码检视3
kernels/bev_pool_v3/op_host/bev_pool_v3_grad.cpp
@@ -0,0 +67,4 @@
67+ (DOUBLE_BUFFER * channel * 3 + DOUBLE_BUFFER * BLOCK_BYTE_SIZE * 3 + channel * 2 + BLOCK_BYTE_SIZE + 1) *
68+ fp32ByteSize;
69+ 
70+ int32_t rankStep = (ubSize - DOUBLE_BUFFER * RANK_KIND * PRESET_RANK_NUM * fp32ByteSize) / channelUBSize;
zhang_qianqian466月3日评论:

针对当前修复的bug,建议后续提供ut进行看护

likedislike
gitdzy
gitdzy成员
6月5日 评论:

当前bevpoolv3grad算子tiling策略优化后,对泛化性约束范围有影响。之后需要增加ut进行看护,同时更新docs的约束范围对外声明。

gitdzy
gitdzy成员
6月5日 评论:

此外,当前bevpoolv3grad算子缺少interShape和interDataType,且tiling侧的模板参数没有意义,需后续修复。

gitdzygitdzy成员
6月5日 预合并成功(commit_id: 734d4b943cde705f016dca55387f7856f6825031)
gitdzygitdzy成员
6月5日 强制推送  1 个提交:bfd16120-fix A2 bevpoolv3 bug with large channel
gitdzygitdzy成员
6月5日 预合并成功(commit_id: 7b30620f11c7d86d955206741b65ae96c3e43a75)
ascend-robotascend-robot成员
6月5日 删除了label:ci-pipeline-passed
AtlasAccountAtlasAccount成员
6月5日 添加了label:ci-pipeline-failed
ascend-robot
ascend-robot成员
6月5日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
6月5日 删除了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
6月5日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
6月5日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
gitdzy
gitdzy成员
6月5日 评论:

compile

likedislike
ascend-robotascend-robot成员
6月5日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
6月5日 添加了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
6月5日 删除了label:ci-pipeline-failed
ascend-robotascend-robot成员
6月5日 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
6月5日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
6月5日 删除了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
6月5日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
6月5日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
6月5日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
6月5日 添加了label:docs-ci-pipeline-success
gitdzygitdzy成员
6月5日 解决了最后一个问题
ascend-robotascend-robot成员
6月5日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
6月5日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
6月5日 评论:
流水线 PR-pipeline_DrivingSDK#1301 [ commitID:bfd16120 ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
恶意代码检查 Antipoison ✅ >>>
编码安全与规范检查 CodeCheck ✅ >>>
CodeCheck_pre-commit ✅ >>>
开源片段检查 SCA ✅ >>>
开发者测试 UT_ARM ✅ >>>
流水线 PR-pipeline_DrivingSDK ✅ >>>
此流水线已支持下列评论快捷指令,仅PR创建者评论有效
  • compile : 运行流水线
likedislike
jayhua
jayhua成员
6月5日 评论:

/approve

likedislike
ascend-robotascend-robot成员
6月5日 添加了label:approved
gitdzygitdzy成员
6月5日 删除了关联的issue:[Roadmap] DrivingSDK Roadmap 2026 Q2
gitdzygitdzy成员
6月5日 关联了issue:[Bug]: A2 BEVPoolv3算子在channel>336时会报错the ub address out of bounds
zhang_qianqian46成员
6月5日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
6月5日 添加了label:lgtm
ascend-robotascend-robot成员
6月5日 合入了pull request
ascend-robot
ascend-robot成员
6月5日 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike