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

Questions on the README - quantization #1

Open
mkvenkit opened this issue Jan 25, 2021 · 1 comment
Open

Questions on the README - quantization #1

mkvenkit opened this issue Jan 25, 2021 · 1 comment

Comments

@mkvenkit
Copy link

mkvenkit commented Jan 25, 2021

Hello, thanks for this very interesting project! Just wanted clarification on these two statements from your readme:

  1. arm_sign_nb_45: ...This version runs but there is an issue with the way the weights were quantized and stored in the weights header.
  2. quantization.ipynb: ...Note that the implementation of quantization in this code is incorrect

I was able to build your example on an nRF52840. So my question is whether the above two issues are still unresolved.

@IngyN
Copy link
Collaborator

IngyN commented Jan 30, 2021

Hi, I was able to get the project to build and run with the libraries and on the chip but the network didn't run properly, it ran with no errors but it didn't recognize the pictures like it should have based on the inference (I was using a pretrained version and loading the weights directly). I didn't have time to debug and figure out exactly why but I believe it might be how I set up / "quantized" the network weights in the quantization file.

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