-
Notifications
You must be signed in to change notification settings - Fork 18
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
dynamic thresholds #2
Comments
Hi
Dynamic upper and lower threshold will update during training, as shown in
.ipynb file.
You can check the implement of original DAC[Chang_2017_ICCV] algorithm.
We share the same threshold mechanism.
Liang Chen <[email protected]> 於 2020年2月10日 週一 上午10:59 寫道:
… hello, I have some question about u and v in your code. You say they are
dynamic in your paper, but it seems that they are fixed in your code.
Therefore, can you supply the dynamic version to me? Thanks!!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#2?email_source=notifications&email_token=ABKMZG3TBDPKE7MVRQUPK5TRCF2URA5CNFSM4KSQY6JKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMJPNQQ>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABKMZG2AS6VGA62LVERVJH3RCF2URANCNFSM4KSQY6JA>
.
|
oh oh, thanks for your reply. I see the dynamic setting in experiment.py file. Then I have another question: above codes seem to indicate that the cross entropy is still restricted in known true labels, but does not support from these high similar pair points. This is inconsistent with the definition of Rij in your self-supervised method. Because I can not find that you will change the labels during training process in experiment.py file. Do I get it wrong? Thanks!!! |
pos_mask and neg_mask response for calculating loss in self-supervised step
(applied on [predicted] sim_mat)
you can print the variables during trining for sanity check.
Liang Chen <[email protected]> 於 2020年2月13日 週四 上午5:36寫道:
… oh oh, thanks for your reply. I see the dynamic setting in experiment.py
file. Then I have another question:
label_mat[labels==-1, :] = -1
label_mat[:, labels==-1] = -1
label_mat[label_mat==0] = 0
label_mat[label_mat==1] = 1
pos_mask = (sim_mat > u_threshold).type(torch.cuda.FloatTensor)
neg_mask = (sim_mat < l_threshold).type(torch.cuda.FloatTensor)
pos_mask[label_mat==1] = 1
neg_mask[label_mat==0] = 1
pos_entropy = -torch.log(torch.clamp(sim_mat, eps, 1.0)) * pos_mask
neg_entropy = -torch.log(torch.clamp(1-sim_mat, eps, 1.0)) * neg_mask
loss = pos_entropy.mean() + neg_entropy.mean() + u_threshold - l_threshold
above codes seem to indicate that the cross entropy is still restricted in
known true labels, but does not support from these high similar pair
points. This is inconsistent with the definition of Rij in your
self-supervised method. Because I can not find that you will change the
labels during training process in experiment.py file. Do I get it wrong?
Thanks!!!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#2?email_source=notifications&email_token=ABKMZGZQ2YSGGIJGUVZQQGLRCVEFZA5CNFSM4KSQY6JKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELU7CXA#issuecomment-585757020>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABKMZGYGJDIM33DQN7DGKGDRCVEFZANCNFSM4KSQY6JA>
.
--
林廷恩 Ting-En (Tony) Lin, Master
Department of Computer Science and Technology
Tsinghua University, Beijing, China, 100084
E-mail: [email protected]
|
Yes, you are right, I misunderstand these two sentences: pos_mask[label_mat==1] = 1 |
Hi, I find that the choice of upper threshold u and lower threshold l in dynamic process has a significant impact on clustering results. The 0.95 and 0.455 are selected by experience or after tuning the network parameter. Have you considered designing a data based selection method for them? For example, based on the similarity matrix. Looking forward to your reply, thanks!!!(Haha, I find that you are in Tsinghua University, I am in Peking University, may I have your WeChat?) |
Sure, my wechat is s950123
Liang Chen <[email protected]> 於 2020年3月11日 週三 上午1:40 寫道:
… Hi, I find that the choice of upper threshold u and lower threshold l in
dynamic process has a significant impact on clustering results. The 0.95
and 0.455 are selected by experience or after tuning the network parameter.
Have you considered designing a data based selection method for them? For
example, based on the similarity matrix. Looking forward to your reply,
thanks!!!(Haha, I find that you are in Tsinghua University, I am in Peking
University, may I have your WeChat?)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#2?email_source=notifications&email_token=ABKMZG6IDPUYEXDDC63ZBATRGZ3QZA5CNFSM4KSQY6JKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOMNWAQ#issuecomment-597220098>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABKMZG6JWCOFEDC57GAGJYLRGZ3QZANCNFSM4KSQY6JA>
.
|
hello, I have some question about u and v in your code. You say they are dynamic in your paper, but it seems that they are fixed in your code. Therefore, can you supply the dynamic version to me? Thanks!!!
The text was updated successfully, but these errors were encountered: