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

Some minor mistakes in the paper #65

Open
Alphacch opened this issue Sep 8, 2023 · 3 comments
Open

Some minor mistakes in the paper #65

Alphacch opened this issue Sep 8, 2023 · 3 comments

Comments

@Alphacch
Copy link

Alphacch commented Sep 8, 2023

Hello, thank you very much for your work. I would like to point out a small bug I found to you, there is a little problem with the first layer convolution of your network parameter picture. It wasn't until I checked your github that I discovered that it was a careless mistake in your paper. Before that, I spent a lot of time calculating the reason...
4518c0f553475365c5125fa56671965
ca016af4ccf68bc03e55bc3a6344974

@Alphacch
Copy link
Author

Alphacch commented Sep 8, 2023

Before this, I analyzed whether there was a problem with the results or the convolution parameters. Or whether there are any subtle differences in using different frameworks, etc.

@forresti
Copy link
Owner

forresti commented Sep 8, 2023

I’m sorry about that! Glad you were able to figure it out. Good detective work. :)

@Alphacch
Copy link
Author

Alphacch commented Sep 8, 2023

#65 (comment)
thank you for your reply. Of course I've been using your squeezenet and I think it's great! Because caffe has stopped maintenance, I recently chose pytorch to build the network, so I discovered your problem. Thanks again for getting back to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants