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.

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

[bug] Using UDP communication, some statements fail to match because line breaks are not cleanly filtered. #171

Open
8 tasks done
zymouse opened this issue Dec 14, 2023 · 4 comments

Comments

@zymouse
Copy link

zymouse commented Dec 14, 2023

settled

  • brach: ros2
  • commit: 861323c
  • Already read the document
  • Debugging the code to make sure it's a code bug
  • Tested TCP and UDP against each other: no problems with TCP, problems with UDP line breaks `\r
  • Grab packets via wireshark:
    • Make sure there are no problems with the data
    • Determine that TCP and UDP are receiving the same data
image ## Description of the problem UDP test: match failed because a statement line break was not filtered cleanly

r'(^\$GP|^\$GN|^\$GL|^\$IN).*\*[0-9A-Fa-f]{2}$', nmea_sentence):

image

Possible causes

data_list = data.decode("ascii").strip().split("\n")

@zymouse
Copy link
Author

zymouse commented Dec 14, 2023

@ericperko extremely grateful

With this modification, the test passes ✅

image

@ericperko
Copy link
Contributor

@zymouse I think you want the current maintainer for this project, @evenator

@evenator
Copy link
Collaborator

@zymouse Your analysis of the situation appears to be correct and I would welcome a pull request to fix the bug.

@zymouse
Copy link
Author

zymouse commented Jan 9, 2024

@evenator #174
It was a pleasure to maintain the program and make him better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants