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

Validation step missing in main_moco.py? #56

Closed
ankuPRK opened this issue Jul 21, 2020 · 1 comment
Closed

Validation step missing in main_moco.py? #56

ankuPRK opened this issue Jul 21, 2020 · 1 comment

Comments

@ankuPRK
Copy link

ankuPRK commented Jul 21, 2020

Hey guys thanks a lot for opening up the MOCO code!

I was going through main_moco.py it doesn't really have a validation step. Does adding a simple validation on pretext task make sense, or do authors feel against it and instead suggest using some downstream task only to make the final decision like the KNN-based monitor, as given in paper for ablation of Shuffle-BN?

Adding a validation step should be easy but it is not there so I was curious/

@KaimingHe
Copy link
Contributor

See #49

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