-
Notifications
You must be signed in to change notification settings - Fork 630
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
centerness shared head #89
Comments
@AllenPeng0209 It is discussed in Section 4.1.2 in our paper. |
Also, would it be better to train 80 centerness scores for 80 class instead of 1 score for 80 class? |
@AllenPeng0209 Sorry, we have not tried this. You might try it by yourself. |
@tianzhi0549 I've tried sharing the center-ness head with the reg head rather than the cls head. This simple modification leads to 0.5 mAP gain for FCOS-R101. You can try this. |
@bearcatt Thank you very much! We will try it. |
@bearcatt We have confirmed it can improve the performance. Do you mind we add this into our code with a proper acknowledgement? |
@tianzhi0549 that's great! You can also refer to the code here. |
@bearcatt Thank you:-). Let me do it. |
@bearcatt Why I share center-ness head in reg branch at mmdetection,it's not improved |
The value of CENTERNESS_ON_REG in the provided yaml file is set to True. |
Hi,
I want to know that is there any experiment results show that centerness branch shared the same parameter with classification branch achieve the best MAP?
Would it be better to separate the centerness branch?
And what it the effect and result behind?
Thanks !
The text was updated successfully, but these errors were encountered: