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

Allowing modification of learning rate for mu #1456

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

XavierAudier
Copy link
Contributor

In Gaussian visible layer, mu should be able to have a learning rate different from 1/sharing.

mu should be able to have a learning rate different from 1/sharing.
@lamblin
Copy link
Member

lamblin commented Mar 26, 2015

Thanks.
Can you add (or expand existing) unit tests to check that the options are correctly recognized and the behaviour is as expected?

@lamblin lamblin self-assigned this Mar 26, 2015
XavierAudier pushed a commit to XavierAudier/pylearn2 that referenced this pull request Mar 27, 2015
@XavierAudier
Copy link
Contributor Author

I've added a test for learning rates : #1460. It makes sure the learning rate for mu is implemented in the same way it is for beta. It runs fine on a dataset but I haven't put that in the test.

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

Successfully merging this pull request may close these issues.

2 participants