Skip to content
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

Did not find the Loss_off defined in Eq(3) #26

Open
zaie opened this issue Apr 26, 2023 · 1 comment
Open

Did not find the Loss_off defined in Eq(3) #26

zaie opened this issue Apr 26, 2023 · 1 comment

Comments

@zaie
Copy link

zaie commented Apr 26, 2023

image
The centernet only returns two outputs, which is different from that described in the paper.
image

Besides, I find the target has key offset. However, it is not used in any loss in the loss dict.
image
image

Is there any reason the Loss_off is omitted or my understanding is wrong.

@AlvinYH
Copy link
Owner

AlvinYH commented Jul 23, 2023

Thanks for your interest in our work. Yes, Loss_off is omitted in our final implementation. Through lots of experiments, considering the speed-performance tradeoffs, we remove the offset branch in HDN and retrain the models. And we'll modify the paper and upload the final version on arXiv. Sorry for the misunderstanding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants