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

It might use unlabeled data to train "fullysup" #6

Open
DoctorKey opened this issue Nov 13, 2018 · 2 comments
Open

It might use unlabeled data to train "fullysup" #6

DoctorKey opened this issue Nov 13, 2018 · 2 comments
Assignees
Labels

Comments

@DoctorKey
Copy link
Contributor

Thank you for your code!
When I run the code by using table-1-cifar10-4000-fullysup.yml, I find it might use unlabeled data to train "fullysup". A batch will contain labeled data and unlabeled data. Since the Wide ResNet has BN layer, it will use unlabeled data to compute the variables of BN.

@avital
Copy link
Contributor

avital commented Nov 13, 2018

Hi @DoctorKey, thanks for the careful analysis. I believe you're right.

We used to have a "supervised_only" flag for this purpose but we removed it to simplify our data loading pipeline.

I wonder if there's a simple change that can resolve this problem, without introducing a full new data loading path...

@avital
Copy link
Contributor

avital commented Nov 13, 2018

Happy to hear any suggestions if you have any. If not, I'll take a closer look at this, but I won't have time to do that this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants