Skip to content

python3 compatible #5

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

python3 compatible #5

wants to merge 1 commit into from

Conversation

iam-prakash
Copy link

Made little corrections to make it python3 compatible and added some formatting.
Please review and merge.

Copy link

@tute tute left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried this, and I get:

Traceback (most recent call last):
  File "pdf417.py", line 312, in <module>
    ret = pdf417_decode(sys.argv[1])
  File "pdf417.py", line 288, in pdf417_decode
    info =  get_cwinfo(codewords)
  File "pdf417.py", line 219, in get_cwinfo
    z = l2%30
TypeError: not all arguments converted during string formatting

@ldtapia
Copy link

ldtapia commented Aug 6, 2020

Hi! I have the same error.. Could you find any fix?

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

Successfully merging this pull request may close these issues.

3 participants