IDEA-Research / MaskDINO

[CVPR 2023] Official implementation of the paper "Mask DINO: Towards A Unified Transformer-based Framework for Object Detection and Segmentation"
Apache License 2.0
1.17k stars 105 forks source link

Should PANO_BOX_LOSS be true for panoptic segmentation? #75

Open jacklishufan opened 1 year ago

jacklishufan commented 1 year ago

It seems that PANO_BOX_LOSS: True will enable various changes described in the paper (no box loss for BG class). However, in the panoptic config, this value is set to False. Is this expected? https://github.com/IDEA-Research/MaskDINO/blob/main/configs/coco/panoptic-segmentation/maskdino_R50_bs16_50ep_3s_dowsample1_2048.yaml#L50

hhaAndroid commented 1 year ago

@jacklishufan I am also very confused. The configuration and code settings are different, but the accuracy is the same as in the paper. It's very strange.