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

Changes for the new MNIST fuel dataset. #618

Merged
merged 3 commits into from
May 4, 2015
Merged

Conversation

Thrandis
Copy link
Contributor

@Thrandis Thrandis commented May 4, 2015

Changes for MNIST refactoring. (mila-iqia/fuel#95)

@vdumoulin
Copy link
Contributor

We'll need to change .travis.yml as well, I'm afraid. Here's what we'll need to do:

  • Change lines 2-4 to
cache
  directories:
    - $TRAVIS_BUILD_DIR/data
  • Remove lines 22-33 (those that download the raw MNIST files)
  • Change line 34 to export FUEL_DATA_PATH=$TRAVIS_BUILD_DIR/data
  • Right after line 40, add
  - |
      if [ ! -f $FUEL_DATA_PATH/mnist.hdf5 ]; then
        cd $FUEL_DATA_PATH
        fuel-download mnist
        fuel-convert mnist
        fuel-download --clear mnist
        cd -
      fi

@@ -29,9 +29,9 @@ def main(save_to, num_epochs, bokeh=False):
weights_init=IsotropicGaussian(0.01),
biases_init=Constant(0))
mlp.initialize()
x = tensor.matrix('features')
x = tensor.tensor4('features')
Copy link
Contributor

Choose a reason for hiding this comment

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

There's an even simpler way to make sure inputs are flat: pass flatten=('features',) to the MNIST constructor.

@vdumoulin
Copy link
Contributor

To fix the failing doctest in docs/index.rst, just change line 89 to

>>> mnist_train = MNIST("train", flatten=('features',))

@Thrandis
Copy link
Contributor Author

Thrandis commented May 4, 2015

mhh some more doc to fix!

vdumoulin added a commit that referenced this pull request May 4, 2015
Changes for the new MNIST fuel dataset.
@vdumoulin vdumoulin merged commit 678ab0e into mila-iqia:master May 4, 2015
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

Successfully merging this pull request may close these issues.

2 participants