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

target logit and its curves #8

Closed
xuezhongcailian opened this issue Feb 11, 2018 · 5 comments
Closed

target logit and its curves #8

xuezhongcailian opened this issue Feb 11, 2018 · 5 comments

Comments

@xuezhongcailian
Copy link

hi,I read your paper and your target logit curves code ,I feel puzzled,your paper say that Wf is also called as the target logit ,but I feel that your target logit curves is not correspond with the definied Wf,it seems not to think about the f.

@happynear
Copy link
Owner

happynear commented Feb 11, 2018

The definition of logit is from this paper: https://arxiv.org/pdf/1701.06548.pdf (z_i in Section 3). It originally means the values that are fed into Softmax loss, and we define the target logit as the logit which is corresponding to the ground truth label.

In the original Softmax, the target logit is W_y^T f. In AM-Softmax, we normalize the W and f and modify the target logit to be s*(W_y^T f - m). Here y is the ground-truth label of the optimizing sample.

@xuezhongcailian
Copy link
Author

hi,I read your target logit curves matlab code on your github draw_psi_curve.m ,I always feel the Angular softmax and AM softmax curves plot the ψ(θ).but conventional softmax plot cos(θ), the target logit is W_y^T f.Angular softmax target logit ||f|| ψ(θ),I feel not corresponding.

@happynear
Copy link
Owner

You are right. I will change the y-label to ψ(θ) in the final version of my paper.

@xuezhongcailian
Copy link
Author

I do not know whether arcface target logit curves have also the problem.

@happynear
Copy link
Owner

happynear commented Feb 23, 2018

Yeah... ArcFace shows a similar figure, and suffers from the same problem.

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