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

A question about the initialization of memory M. #8

Closed
HX-idiot opened this issue Jun 16, 2019 · 2 comments
Closed

A question about the initialization of memory M. #8

HX-idiot opened this issue Jun 16, 2019 · 2 comments
Labels
question Further information is requested

Comments

@HX-idiot
Copy link

Hi, brother, it was great to have a look at your paper recently,
what i confused is ,at the beginning of stage 2, you use average representation for each class as memory, how about just using the fc parameter trained in stage_1?(we can also use cosine classifier in stage_1) Maybe it's more conclusive? Does there some difference between the two ways?
Thanks~

@zhmiao
Copy link
Owner

zhmiao commented Jun 17, 2019

Hello @HX-idiot, thanks for asking. Actually, there could be many different ways to construct the memory bank. Feature centroids is one of the most straightforward ways because it is easy to implement and can capture the common invariance within each class that represents the whole class pretty well. At the same time, it fits the use of center loss that makes class clusters more compact and pushes the centroids further away from each other. Weight average is another way for sure. We did not test different types of memories in our paper, but should be an interesting thing to do.

The reason why we did not apply cosine classifier in the first stage was that the main purpose of the first stage was to initialize the memory bank. We believe that applying cosine classifier at the second stage training is enough for the final classification. However, to further improve the performance, applying cosine to the first stage might help as well.

@zhmiao zhmiao added the question Further information is requested label Jun 17, 2019
@zhmiao
Copy link
Owner

zhmiao commented Jul 15, 2019

Since it has been over 20 days, I am closing this issue for now. If you have any more questions, we will reopen this issue. Thank you very much.

@zhmiao zhmiao closed this as completed Jul 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants