luxiin

It helps grid branch more robust to diverse proposals.

@foralliance that's right.

Only cls-loss. We simply discard reg-loss by overriding loss function in BBoxHead.

https://github.com/STVIR/Grid-R-CNN/blob/23ef988911b3b68fe8d5b21225995d6b392ee7b6/mmdet/models/bbox_heads/convfc_bbox_head.py#L218

@zimenglan-sysu-512 We override the loss function of BBoxHead, it's original version contains reg loss like this: https://github.com/STVIR/Grid-R-CNN/blob/23ef988911b3b68fe8d5b21225995d6b392ee7b6/mmdet/models/bbox_heads/bbox_head.py#L103

In mmdetection, one can disable the reg-loss explicit in config file like this: https://github.com/open-mmlab/mmdetection/blob/713e98bc1bfb842760df6be73b9520ee775d3c06/configs/grid_rcnn/grid_rcnn_gn_head_r50_fpn_2x.py#L36 The feature is not implemented in this repo.

Hi, @zimenglan-sysu-512 . The codes you point out are the implementation of Grid Point Specific Representation Region proposed in Grid R-CNN Plus. You can refer to section 3.1 in [https://arxiv.org/pdf/1906.05688.pdf](url)...