Skip to content

Fix JTOP info exception for python3.9/python3.11#309

Merged
rbonghi merged 3 commits into
rbonghi:developfrom
RaymondCM:patch-1
Jan 3, 2023
Merged

Fix JTOP info exception for python3.9/python3.11#309
rbonghi merged 3 commits into
rbonghi:developfrom
RaymondCM:patch-1

Conversation

@RaymondCM
Copy link
Copy Markdown
Contributor

.tostring() is deprecated, .tobytes() can be used instead.

Otherwise in Python3.11 the following error is raised on the info page of JTOP:

AttributeError: 'array.array' object has no attribute 'tostring'

dependabot Bot and others added 2 commits November 3, 2022 17:07
Bumps [dependabot/fetch-metadata](https://github.com/dependabot/fetch-metadata) from 1.3.4 to 1.3.5.
- [Release notes](https://github.com/dependabot/fetch-metadata/releases)
- [Commits](dependabot/fetch-metadata@v1.3.4...v1.3.5)

---
updated-dependencies:
- dependency-name: dependabot/fetch-metadata
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
.tostring() is deprecated, .tobytes() can be used instead.
@RaymondCM
Copy link
Copy Markdown
Contributor Author

After inspection, #153 also resolves this and raises another issue with python2.7 compatibility - therefore I propose the following 2.7/3.X compatible version. Interested to see if tests pass once approved.

@rbonghi rbonghi changed the base branch from master to develop January 3, 2023 14:20
@rbonghi
Copy link
Copy Markdown
Owner

rbonghi commented Jan 3, 2023

Hi @RaymondKirk ,

I am sorry for this late reply.
Thank you very much for your message! I will push your fix for the next release!

I am also fixing the CI, I noticed some bugs

@rbonghi rbonghi merged commit 6769b4b into rbonghi:develop Jan 3, 2023
@rbonghi rbonghi added the bug Something isn't working label Jan 3, 2023
@rbonghi rbonghi added this to the 4.0.0 milestone Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants