Decoding on MacOS gives raw data about pylibdmtx HOT 8 CLOSED

I have investigated a little more deeply and have come to the following conclusions:

1. This is not a platform-dependent bug
2. This is a problem with pylibdmtx, not the upstream library

Evidence

This is not a platform independent bug

I have managed to replicate this problem on my machine (Debian testing) with the apt installed libdmtx (v0.7.5), as has @akhmerov.

This is a problem with pylibdmtx

I used the first part of @justinvdk's script above to generate a problematic datamatrix. I then saved this to a PNG file and read it in using the dmtxread program installed with apt, which is linked against libdmtx v0.7.5, and it produces the correct result.

What's the problem

I believe that the API to libdmtx has changed between 2 bugfix releases, when really there should have been a minor (or even major) version release.

I used the shell script from this gist with git bisect in the libdmtx repo to find that this commit is where things stopped working. I notice that this commit actually changed the API of libdmtx by adding extra fields to some of the exported datastructures. This means that the wrappers in wrapper.py are formally incorrect, and things break in unexpected ways.'

from pylibdmtx.

Included in v.0.1.8, now on PyPI.

from pylibdmtx.

from pylibdmtx.

Seems like this was not fixed after all... I am also getting these unexpected results on MAC

from pylibdmtx.

Downgrading to libdmtx to version 0.7.4 fixes the issue. Therefore I suggest the issue is with latest libdmtx and not pylibdmtx.

from pylibdmtx.

I now observe this issue with libdmtx 0.7.5 and Ubuntu 18.10.

from pylibdmtx.

Hi @quicklizard99 @kursancew, have you had a chance to look at this? This seems to be a breaking issue on fresh installs of this repo. @jbweston's fix resolves it on my machine. His PR is ready to be merged, but additional information is needed on how to update CI to run with the newest version of libdmtx.

(Although, some broader changes may be required if backwards compatibility with libdmtx is viewed as a necessity. Depending on scope, I can potentially help out as well in order to get this repo working again.)

from pylibdmtx.

Awesome, thank you!

from pylibdmtx.