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

Add AlexNet on CIFAR10 #335

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

Conversation

francescodisalvo05
Copy link

Hello everyone!

I recently completed to work on a tutorial regarding the implementation of AlexNet on Cifar10.

There is a minor difference with the "real" implementation due to the dimension of the images. You can have a look here: #334

Unfortunately I have no GPU on my laptop therefore I couldn't test the CUDA version. For that reason I avoided to propose also its code without a confirmation that it works.

Copy link
Member

@ToucheSir ToucheSir left a comment

Choose a reason for hiding this comment

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

Thanks! A quick first review pass:

vision/alexnet_cifar10/README.md Outdated Show resolved Hide resolved
vision/alexnet_cifar10/alexnet_cifar10.jl Outdated Show resolved Hide resolved
vision/alexnet_cifar10/alexnet_cifar10.jl Outdated Show resolved Hide resolved
vision/alexnet_cifar10/alexnet_cifar10.jl Outdated Show resolved Hide resolved
vision/alexnet_cifar10/alexnet_cifar10.jl Outdated Show resolved Hide resolved
@CarloLucibello
Copy link
Member

this PR seems good, but do we need it? We already have an example with vgg on cifar10 and we don't want all combinations of models and datasets

@mcabbott mcabbott added the new model new arrival in the zoo label Dec 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new model new arrival in the zoo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants