-
Notifications
You must be signed in to change notification settings - Fork 2.8k
CNN Architecture implementations in Tensorflow #17
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
Conversation
Hey that's awesome @the-robot! I just looked through it rather quickly but I think you could make different VGG types a little bit cleaner perhaps, take a look at my PyTorch implementation: Machine-Learning-Collection/ML/Pytorch/CNN_architectures/pytorch_vgg_implementation.py Lines 16 to 62 in 79f2e19
I think doing something like that with the VGG_types could make it a little more compact |
Thanks for the suggestion. Initially I made it that way to group some layers together by following ConvNet config table from original paper. But storing different types in a dictionary would be much cleaner. I also removed training part and put |
Added links in the readme, read through the implementations and looked really nice, liked the implementation of ResNet 👍 |
Your YT contents are awesome!! Especially those CNN architectures from scratch videos. I myself is trying to learn DL and your videos helped me understand the concept better when I was reading the academic papers.
Not very long ago, I started implementing some of the popular CNN architectures with Tensorflow 2.0 in my repo and I think it would be good to PR those to here so the rest can checkout both PyTorch and Tensorflow implementations.
I have implemented