wusize / ovdet

[CVPR2023] Code Release of Aligning Bag of Regions for Open-Vocabulary Object Detection
https://openaccess.thecvf.com/content/CVPR2023/papers/Wu_Aligning_Bag_of_Regions_for_Open-Vocabulary_Object_Detection_CVPR_2023_paper.pdf
Other
172 stars 4 forks source link

why use two inputs branch (det and baron_kd) #26

Closed qsunyuan closed 1 year ago

qsunyuan commented 1 year ago

Hi!

https://github.com/wusize/ovdet/blob/c67eecf11db35a7cbe020298572654506cfdff3e/ovdet/models/detectors/two_stage.py#L52

Why do we need two branches for forward propagation? It seems that one branch can calculate det-loss and baron-kd loss and reduce the amount of calculation.

wusize commented 1 year ago

This is to make the code base more flexible. Disentangling the det and ovd branch can make the code base suitable for any OVD methods.