Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

rostopic ignores 64-bit values when printing in plotting friendly format#1096

Closed
mstegmaier wants to merge 2 commits intoros:lunar-develfrom
mstegmaier:lunar-devel
Closed

rostopic ignores 64-bit values when printing in plotting friendly format#1096
mstegmaier wants to merge 2 commits intoros:lunar-develfrom
mstegmaier:lunar-devel

Conversation

@mstegmaier
Copy link
Copy Markdown

Fixes #1095.

@mstegmaier mstegmaier changed the title Fixes #1095. rostopic ignores 64-bit values when printing in plotting friendly format Jul 11, 2017
@dirk-thomas
Copy link
Copy Markdown
Member

long doesn't exist in Python 3. Please update the patch to work with both Python 2 as well as 3.

@dirk-thomas
Copy link
Copy Markdown
Member

In order to simplify the patch I would suggest to define long as being the same as int in the case of Python 3. That way the conditions stay shorter, more readable and less repetitive.

@dirk-thomas
Copy link
Copy Markdown
Member

Please see #1110 for a different patch to address the problem.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants