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

DBM - Hinton & Salakhutdinov paper #1239

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

DBM - Hinton & Salakhutdinov paper #1239

wants to merge 32 commits into from

Conversation

carloderamo
Copy link

Pylearn2 implementation of the whole procedure to train a DBM described by Hinton & Salakhutdinov in "An Efficient Learning Procedure For Deep Boltzmann Machines".
This pr adds a fast version for the test script to be run on travis together with some changes to the format and the readme file. I also moved augment_input from datasets to /scripts/dbm.

@dwf
Copy link
Contributor

dwf commented Feb 11, 2015

@carloderamo Can you rebase and squash this into a single commit?

As this is all concerning DBM stuff that nobody knows anything about other than @goodfeli I'm going to punt the review to him.

from pylearn2.utils import serial


class MNIST_AUGMENTED(DenseDesignMatrix):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should not need a new Dataset object to do this.

@goodfeli
Copy link
Contributor

Since I've requested relatively major changes based on the small parts I have reviewed, I'm going to quit reviewing early and will review the rest if / when the first changes are made.

@carloderamo
Copy link
Author

Currently I have a lot of work to be done for completing my MSc thesis on
May. I'll work on my pull request, making another one with all the changes
requested, as soon as I'll finish it.
Best regards.

Carlo D'Eramo

Since I've requested relatively major changes based on the small parts I
have reviewed, I'm going to quit reviewing early and will review the rest
if / when the first changes are made.


Reply to this email directly or view it on GitHub
#1239 (comment).

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.

3 participants