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.

Sign up for GitHub

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

Jump to bottom

"TypeError: ord() expected string of length 1, but int found" - python3 (TinyFPGA BX, Ubuntu 19.04) #48

Closed
zrecore opened this issue May 13, 2019 · 5 comments · Fixed by #49
Closed

"TypeError: ord() expected string of length 1, but int found" - python3 (TinyFPGA BX, Ubuntu 19.04) #48

zrecore opened this issue May 13, 2019 · 5 comments · Fixed by #49

Comments

@zrecore
Copy link

zrecore commented May 13, 2019

TinyFPGA-Bootloader/programmer/tinyprog/__init__.py

Line 38 in 99f87a5

output += " ".join(["%02x" % ord(x) for x in data[i:i + 16]]) + "\n"

... Removing the call to ord() here allows the script to continue execution.
mithro added a commit to mithro/TinyFPGA-Bootloader that referenced this issue May 14, 2019
@mithro
Make sure pretty_hex works under Python 2 and Python 3.
1093b92 
 * Run doctests under tox.
 * Add pretty_hdex doc tests for string, bytes and unicode

Fixes tinyfpga#48
mithro added a commit to mithro/TinyFPGA-Bootloader that referenced this issue May 14, 2019
@mithro
Make sure pretty_hex works under Python 2 and Python 3.
dd3c01f 
 * Run doctests under tox.
 * Add pretty_hdex doc tests for string, bytes and unicode

Fixes tinyfpga#48
@mithro mithro mentioned this issue May 14, 2019
Merged
mithro added a commit to mithro/TinyFPGA-Bootloader that referenced this issue May 14, 2019
@mithro
Make sure pretty_hex works under Python 2 and Python 3.
1d9bfd7 
 * Run doctests under tox.
 * Add pretty_hex doctests for string, bytes and unicode types.

Fixes tinyfpga#48
@mithro
Copy link
Collaborator

mithro commented May 14, 2019

@zrecore Can you please test this fixed your issue?

@mithro mithro reopened this May 14, 2019
@zrecore
Copy link
Author

zrecore commented May 14, 2019

@mithro ok, testing now

@zrecore
Copy link
Author

zrecore commented May 15, 2019

Ok, no more python errors using mithro/TinyFPGA-Bootloader repo, master branch. 👍
(Still getting the gateware size issue, but that's probably a different problem)

Screen Shot 2019-05-14 at 5 17 13 PM

@mithro
Copy link
Collaborator

mithro commented May 15, 2019

@zrecore It looks like your still failing? Can you log a full bug for that somewhere (maybe on https://github.com/timvideos/litex-buildenv/issues)?

@zrecore
Copy link
Author

zrecore commented May 15, 2019

@mithro added issue at timvideos/litex-buildenv#137

@zrecore zrecore closed this as completed May 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make sure pretty_hex works under Python 2 and Python 3. mithro/TinyFPGA-Bootloader
2 participants
@mithro @zrecore