Skip to content

fixed bev_pool cuda .cu wrong cur_geom_feats address in bev_pool_cuda.cu#3143

Open
ZouJiu1 wants to merge 1 commit intoopen-mmlab:mainfrom
ZouJiu1:main
Open

fixed bev_pool cuda .cu wrong cur_geom_feats address in bev_pool_cuda.cu#3143
ZouJiu1 wants to merge 1 commit intoopen-mmlab:mainfrom
ZouJiu1:main

Conversation

@ZouJiu1
Copy link
Copy Markdown

@ZouJiu1 ZouJiu1 commented Apr 19, 2026

Contribution

I fix a problem with wrong cur_geom_feats address in file of bev_pool_cuda.cu.

Problem

The result is different between the output of function QuickCumsumCuda and the output of function QuickCumsum in file bev_pool.py and another repo.

the reason is the wrong implement of file bev_pool_cuda.cu and another repo

Reproduce

To reproduce the wrong different result, you can follow the instruction of file README.md https://github.com/ZouJiu1/bevPool/blob/master/README.md

step 1

reproduce the problem

download the file bev_pool_cuda.cu

mv bev_pool/src/bev_pool_cuda.cu bev_pool/src/bev_pool_cuda_tmp.cu

git clone https://github.com/ZouJiu1/bevPool.git

cd bevPool

pip install -e .

cd bev_pool

python3 bev_pool.py

by checking or debuging or printing the variables like error, errorMean, www, tmp, tmp2 in file bev_pool.py, you can reproduce the problem

step 2

fix the problem

mv bev_pool/src/bev_pool_cuda_tmp.cu bev_pool/src/bev_pool_cuda.cu

cd bevPool

pip install -e .

cd bev_pool

python3 bev_pool.py

by checking variables in file bev_pool.py, you will find the problem is fixed.

Environment:

Ubuntu22.04
Python 3.10.12
torch 2.2.2+cu118
torchvision 0.17.2+cu118
mmcv 2.1.0
mmengine 0.10.7
mmdet 3.3.0
mmdet3d 1.4.0
cuda 11.8
cudnn 8.9.7
gcc g++ 11.4.0

Motivation

fix some problem.

Modification

fixed wrong cur_geom_feats address in file of bev_pool_cuda.cu

others:

hustvl/MapTR#212
autowarefoundation/autoware_universe#12491
#3143
weiyangdaren/Fisheye3DOD#4
hustvl/VMA#21

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 19, 2026

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants