Adding Stacked_MNIST #230

Open josemanuel22 opened this issue 1 year ago • 2 comments

May 30 '24 10:05 josemanuel22

Thanks for this PR. You need to include the file in src/MLDatasets.jl. Also needs some tests, a docstring and an entry in the documentation in docs/src/datasets

Jun 27 '24 09:06 CarloLucibello

I have added the documentation and the tests associated with the new StackedMnist dataset. The tests are failing, showing the following error:

  Got exception outside of a @test
  SystemError: opening file "/home/runner/work/MLDatasets.jl/MLDatasets.jl/test/datasets/vision/stacked_mnist.jl"

I don't understand this, as they passed locally. Other than that, I think the PR is ready.

Aug 16 '24 16:08 josemanuel22