Skip to content

fixed bev_pool cuda .cu wrong cur_geom_feats address in file bev_pool_cuda.cu#212

Open
ZouJiu1 wants to merge 1 commit intohustvl:mainfrom
ZouJiu1:main
Open

fixed bev_pool cuda .cu wrong cur_geom_feats address in file bev_pool_cuda.cu#212
ZouJiu1 wants to merge 1 commit intohustvl: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 or 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:

#212
autowarefoundation/autoware_universe#12491
open-mmlab/mmdetection3d#3143
weiyangdaren/Fisheye3DOD#4
hustvl/VMA#21

Signed-off-by: ZouJiu1 <1069679911@qq.com>
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.

1 participant