open-mmlab / mmdetection

OpenMMLab Detection Toolbox and Benchmark
https://mmdetection.readthedocs.io
Apache License 2.0
29.71k stars 9.48k forks source link

the min_overlap(0.3) is different from CenterNet Code(0.7) #8036

Open Andrewymd opened 2 years ago

Andrewymd commented 2 years ago

https://github.com/open-mmlab/mmdetection/blob/4bd4ace53a626659070204236da5b8e14ae9cd08/mmdet/models/dense_heads/centernet_head.py#L229 Hi, I do not understand why change the min_overlap value, Could you tell me why?

hhaAndroid commented 2 years ago

@youmingdeng should be consistent with the paper. Please post the details, thanks

Andrewymd commented 2 years ago

This is the repo of the CenterNet, https://github.com/xingyizhou/CenterNet/blob/2b7692c377c6686fb35e473dac2de6105eed62c6/src/lib/utils/image.py#L95 the paper does not mention the min_overlap, I just reference the code of the author.

batman47steam commented 1 year ago

I found this too, It seems that in cornernet and centernet, they all use the min_overlap of 0.7