aim-uofa / AdelaiDet

AdelaiDet is an open source toolbox for multiple instance-level detection and recognition tasks.
https://git.io/AdelaiDet
Other
3.37k stars 647 forks source link

The formulation of training regression targets in FCOS paper #161

Open geoexploring opened 4 years ago

geoexploring commented 4 years ago

Hi, thank you all for making FCOS available!

Two things in the FCOS paper confuse me:

  1. As shown in the figure below, is there a comma missing in the red line? I found this in both the conference edition and the journal edition of FCOS paper. 0

  2. As the FCOS paper says, 1 and, the training regression targets for the location is formulated as: 2 However, when I read the FCOS code, I found it was not the same as the definition in the paper, https://github.com/aim-uofa/AdelaiDet/blob/c07ba9a6aabfc108b4e3c7b7c31b05e069d9cdc7/adet/utils/comm.py#L54 https://github.com/aim-uofa/AdelaiDet/blob/c07ba9a6aabfc108b4e3c7b7c31b05e069d9cdc7/adet/modeling/fcos/fcos_outputs.py#L191 https://github.com/aim-uofa/AdelaiDet/blob/c07ba9a6aabfc108b4e3c7b7c31b05e069d9cdc7/adet/modeling/fcos/fcos_outputs.py#L208 The definition in the code feels more like the following formula: 3

Please correct me if I made any mistakes. Thank you in advance

tianzhi0549 commented 4 years ago

@geoexploring 1) Thank you for pointing it out. You are right. We miss a comma here. 2) The implementation is consistent with our paper. locations are x and y in Eq (1).

geoexploring commented 4 years ago

@tianzhi0549 Thank you for your reply!

To 2), my understanding is,

I feel that Eq (1) is subtracting the numbers from two different coordinate systems.

Please correct me if I made any mistakes. Thank you in advance

tianzhi0549 commented 4 years ago

@geoexploring It seems you are right ... We mix up the two coordinate systems here. Sorry about that. We will fix it in a revision.

geoexploring commented 4 years ago

@tianzhi0549 hahaha, thank you for your honesty! many thanks!