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

a mistake in the paper #1

Open
zw1992 opened this issue May 9, 2019 · 4 comments
Open

a mistake in the paper #1

zw1992 opened this issue May 9, 2019 · 4 comments

Comments

@zw1992
Copy link

zw1992 commented May 9, 2019

large

the output channel is mismatch with the output size ,maybe it should be 112.

@kuan-wang
Copy link
Owner

Thanks a lot! I've just updated the codes.
The gap of the model size is still large, welcome to report more bugs :)

@triangleCZH
Copy link

According to the paper right now, I believe they should be both 160 instead.
p.s. The paper has fixed the stride error in V3-small

@ujsyehao
Copy link

@kuan-wang
Hi, what are your mobilenet v3 small 1.0 params?
I calculate it and get 3.09 million (not include BN params), the paper claims 2.9 Million.

@kuan-wang
Copy link
Owner

The paper still has some bugs, now I renew the codes and the model size is close to paper.

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

4 participants