Skip to content
This repository was archived by the owner on Sep 13, 2023. It is now read-only.

Alex Yang CV Bootcamp#109

Open
YMRTZ wants to merge 6 commits into
UWARG:masterfrom
YMRTZ:master
Open

Alex Yang CV Bootcamp#109
YMRTZ wants to merge 6 commits into
UWARG:masterfrom
YMRTZ:master

Conversation

@YMRTZ
Copy link
Copy Markdown

@YMRTZ YMRTZ commented Sep 18, 2022

No description provided.

Copy link
Copy Markdown

@mikashaw mikashaw left a comment

Choose a reason for hiding this comment

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

Hey Alex! Would you mind just adding loss and accuracy plot images and the I can review, thanks :)

Copy link
Copy Markdown

@mikashaw mikashaw left a comment

Choose a reason for hiding this comment

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

Hey Alex, great job on the bootcamp! For this team, we require a 65- 70% min accuracy to have it be considered complete - I would recommend adding some convolutions and max pooling to your neural network! (Try a CNN) - if you need any help, let myself or aydan know!

@YMRTZ
Copy link
Copy Markdown
Author

YMRTZ commented Sep 21, 2022

Using a CNN doesn't seem to have done much for the accuracy. Any tips on how to improve the convolution bit?

Comment thread main.py
logits = self.linearReLUStack(x)
logits = torch.reshape(x, (32,32,3))
logits = logits.permute(2,0,1)
logits = self.conv1(logits)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It looks like your using convolutions without pooling in between, as well as not adding enough convolutiopns for the image features to be extracted. Normally, A typical CNN looks like: Cnv2D -> MaxPool -> Conv2D -> Max Pool -> Flatten -> Dense layers with activation -> final softmax or sigmoid layer. I would try (1) adding max pooling (I see it in the class initialization but not the forward() method), and (2) tweaking the CNN parameters as having only 1 dim in the argument (e.g for your second convolution ) will not do much! Here is a pretty helpful article going over CNN architecture https://medium.com/thecyphy/train-cnn-model-with-pytorch-21dafb918f48 with sample code, you don't need to follow this - it just has some helpful tips about how a typical CNN should look like et cetera!

Copy link
Copy Markdown

@mikashaw mikashaw left a comment

Choose a reason for hiding this comment

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

Hey Alex! I've written a comment for this pull request in the main.py file, hope that helps!

@mgupta27 mgupta27 closed this May 20, 2023
@mgupta27 mgupta27 reopened this May 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants