search

SxJyJay / MSMDFusion

[CVPR 2023] MSMDFusion: Fusing LiDAR and Camera at Multiple Scales with Multi-Depth Seeds for 3D Object Detection
Apache License 2.0
167 stars 10 forks source link

about NMS #43

Open sun-yue2002 opened 2 months ago

sun-yue2002 commented 2 months ago

Hi!

I noticed the nms_type is none in config file. I wonder whether you have done the nms part during the 3Dbboxes generating part.

wish for your response

SxJyJay commented 2 months ago

For nms_type, we follow the configuration of TransFusion. It seems that you do not employ nms operation due to the object query design.

sun-yue2002 commented 2 months ago

For nms_type, we follow the configuration of TransFusion. It seems that you do not employ nms operation due to the object query design.

YES. I figure it out by reading the paper of transufusion. It seems the transfusion head part apply a nms-free strategy. Besides, the AP of ped and traffic_cone nearly didn’t change after I apply circle_nms to bbox of this 2 classes.