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

The problem of the top 1 acc #1

Closed
plzhang4321 opened this issue Dec 23, 2020 · 2 comments
Closed

The problem of the top 1 acc #1

plzhang4321 opened this issue Dec 23, 2020 · 2 comments

Comments

@plzhang4321
Copy link

what means "it may be in std" . Your awesome implementation in pytorch, but I cnnnot understand its big drop~

@senya-ashukha
Copy link
Collaborator

senya-ashukha commented Dec 23, 2020

Hi,

Thank you for the comment!

It means that results can differ depending on a random seed. This is an expensive setup that takes 16v100 and 170h, and we tried it only once. Also, there is a mismatch in default PyTorch and TF augmentation of ImageNet images; in PyTorch, to the best of my knowledge, there is no min_object_covered parameter https://github.com/google-research/simclr/blob/3fb622131d1b6dee76d0d5f6aac67db84dab3800/data_util.py#L269.

Let us know if you have any further questions!

@plzhang4321
Copy link
Author

Thank you~

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

3 participants