Skip to content
This repository has been archived by the owner on Nov 2, 2018. It is now read-only.

Small error in cifar generative L2/L1 regularization #5

Closed
skaae opened this issue Nov 6, 2015 · 1 comment
Closed

Small error in cifar generative L2/L1 regularization #5

skaae opened this issue Nov 6, 2015 · 1 comment

Comments

@skaae
Copy link

skaae commented Nov 6, 2015

I think there is a small error in the L2/L1 regularization in the cifar models.

In the generative opfunc you regularize the discriminative parameters.

https://github.com/facebook/eyescream/blob/master/cifar/train/adversarial.lua#L87

@soumith
Copy link
Contributor

soumith commented Dec 15, 2015

Thanks for finding the bug. Fixed it.

@soumith soumith closed this as completed Dec 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants