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

Momentum Update #57

Open
Macc520 opened this issue Sep 22, 2022 · 2 comments
Open

Momentum Update #57

Macc520 opened this issue Sep 22, 2022 · 2 comments

Comments

@Macc520
Copy link

Macc520 commented Sep 22, 2022

First of all, I would like to say that this is a very milestone work. But I need to solve several problems so that I can better understand it.

  1. In loss_ contras_ mem file, I only see encode_ q. Is it actually encode_ K is not used in the paper.

  2. What is the difference between ContrastAuxCELoss and ContrastCELoss.

Looking forward to your reply. Thank you very much!!!!!!

@tfzhou
Copy link
Owner

tfzhou commented Oct 5, 2022

Hi @Macc520 , 1) we did not use encode_K as has been used in MoCo. Adding another momentum encoder does not lead to obvious gains in performance, but will significantly increase the memory and computation costs. 2) Aux means an auxiliary cross-entropy loss which is added to intermediate features. With it, the final loss will has a form of ce loss + 0.4 * auxiliary ce loss. It is a trick widely used in semantic segmentation training.

@Macc520
Copy link
Author

Macc520 commented Oct 6, 2022 via email

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