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

Student should set to eval() before calculating s_logits_l ? #9

Closed
Vurkty opened this issue Mar 6, 2021 · 3 comments
Closed

Student should set to eval() before calculating s_logits_l ? #9

Vurkty opened this issue Mar 6, 2021 · 3 comments

Comments

@Vurkty
Copy link

Vurkty commented Mar 6, 2021

If you don't do this, model's output will be random because of the dropout etc.
Then your s_loss_l_new and s_loss_l_old will be random too and your dot_production won't give right feedback.
Maybe this issue affects your results

@kekmodel
Copy link
Owner

kekmodel commented Mar 8, 2021

I'll test this. Thank you!

@kekmodel
Copy link
Owner

Test accuracy is 94.0.

@Vurkty
Copy link
Author

Vurkty commented Mar 10, 2021

Well, it seems not work.
Thanks for your test!

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