This repository was archived by the owner on Apr 23, 2025. It is now read-only.
Use Epochs to load CIFAR10 #495
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR starts the work of removing
Batcher
and useEpochs
instead.To avoid one gigantic PR, I have created new protocol named
ImageClassificationData
that I use and kept the old cifar model in OldCifar10 (to use it in the benchmarks). The protocolImageClassificationData
can be renamed once everything is done and the currentImageClassificationDataset
is removed.The OldCifar10 file can be removed when MNIST and the benchmarks have been converted to use Epochs.
New with this implementation: