-
Notifications
You must be signed in to change notification settings - Fork 136
The demo apk does not output correct results #21
Comments
This may be an issue related to the YUV decoding (and improper input being fed into the predictor) based on your particular device. Please pull down the latest version with a fix, which hopefully resolves the issue for you. If that does not work, may I ask what device you are using? |
@bwasti |
I don't know if I am right or not. Maybe (max[_j - 1] = max[_j];) should be changed to (max[_j] = max[_j - 1];) in the native-lib.cpp. |
@tlhchutang Hello, I met the same problem like you.How do you use Opencv's convert function, Can you give it a detailed description? |
@tlhchutang , @bwasti maybe from yuv to rgb conversion code is not correct, I make some change , result seems better https://github.com/northeastsquare/AICamera |
Tested @northeastsquare's change and results seems to be good. Tested on a Motorola G5 Plus (~2.9fps) |
I think the error comes from yuv to rgb conversion and I have clarified it in #14.
so the code in native_cpp should be
And I also change the IMG_W and IMG_H to 224 |
Hi all! Has this bug been fixed? My results are pretty poor with the standard Thanks! |
I directly run the apk build by this project without any change on my own android device.
However, it seems that the outputs(top5 labels) do not make any sense, I also tried to use a better model, that is the GoogleNet converted from the Caffe model zoo using the tool provided by Caffe2, the result is still very strange, none of the top 5 labels is correct at all time.
The text was updated successfully, but these errors were encountered: