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

Model looks unoptimised #10

Open
mrgloom opened this issue Mar 18, 2020 · 1 comment
Open

Model looks unoptimised #10

mrgloom opened this issue Mar 18, 2020 · 1 comment

Comments

@mrgloom
Copy link

mrgloom commented Mar 18, 2020

Screenshot 2020-03-18 at 18 33 00

  1. BN can be fused
  2. batchnorm and scale are separated layers here this is because of bad Caffe implementation.

BTW is there any app where we just can benchmarks some other pertained classification models on iPhone?

@hollance
Copy link
Owner

Yes, you're correct. But this is also an old model so it's not really worth fixing. I'd recommend getting a more up-to-date checkpoint from the TensorFlow models repo instead, and converting that.

(P.S. I believe Core ML automatically does the BN fusing behind the scenes.)

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