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

Fail with the HTML5 File API to decode a pre-taken image #30

Closed
devnev0 opened this issue May 4, 2015 · 5 comments
Closed

Fail with the HTML5 File API to decode a pre-taken image #30

devnev0 opened this issue May 4, 2015 · 5 comments
Labels

Comments

@devnev0
Copy link

devnev0 commented May 4, 2015

If I take a barcode photo with a Moto G 4G, I have any result. When I try with the same image (taken with this phone) on a pc, I view this error:
"Error: Image dimensions do not comply with the current settings: Width (449.38271604938274 )and height (800) must a multiple of 1
https://localhost:8383/OnMyShelves/js/libs/serratus/quagga.min.js
Line 9"

Firstly, I think the image file is corrupted but it works fin with other decode Library.

@serratus
Copy link
Owner

serratus commented May 5, 2015

Thanks for reporting! The main problem is, that the image has really odd dimensions. Can you tell me the original resolution of the file? It seems like the aspect ratio is 16:9, but missing one or two pixels. That's definitely something I have to address, so that QuaggaJS can work with all images, regardless of the size.

I'll try to come up with a solution. In the meantime, can you post the image and/or its dimensions?

@serratus serratus added the bug label May 5, 2015
@devnev0
Copy link
Author

devnev0 commented May 5, 2015

Yes, it 'll be easier for you with the image:
img_20150505_105109950

@serratus
Copy link
Owner

serratus commented May 5, 2015

Thanks a lot for the image. The dimensions are really odd (1.456px × 2.592px) and that's why it does not work with the current implementation. I hope to get around this limitation in the next few days. Basically, your image is not even real 16:9 aspect ratio, and I have to find a way to accept all dimensions, regardless of aspect ratio and internal needs.

@devnev0
Copy link
Author

devnev0 commented May 5, 2015

Yes it's very odd.
Thanks you very much.

@jamsshhayd
Copy link

was this issue fixed?

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

No branches or pull requests

3 participants