Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

about NMS #43

Open
sun-yue2002 opened this issue Apr 7, 2024 · 2 comments
Open

about NMS #43

sun-yue2002 opened this issue Apr 7, 2024 · 2 comments

Comments

@sun-yue2002
Copy link

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
Copy link
Owner

SxJyJay commented Apr 16, 2024

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
Copy link
Author

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.

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

No branches or pull requests

2 participants