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

Communication error using UART to USB #63

Open
Danilrivero opened this issue Apr 20, 2023 · 8 comments
Open

Communication error using UART to USB #63

Danilrivero opened this issue Apr 20, 2023 · 8 comments

Comments

@Danilrivero
Copy link

Hello,

I have bumped into the following issue when trying to connect to the BNO055. I made sure it properly works beforehand by using a small Arduino snippet and it does.

I get the following message when running the node configured with the yaml file.

image

CP2104 and connections work properly and the tty/USB0 port is properly attached:

image

I am using Ubuntu 22.04 with ROS2 Humble and all the libraries required to run the driver are installed.

The only parameter changed is the baudrate due to my requirements:

image

Has anyone bumped into this issue before? Any advice on how to proceed? The response header 0xff does not give me enough information to deal with it.

@github-actions
Copy link

Thank you for creating your first issue on this repo! Give me some time to review and respond to your problem.

@Danilrivero
Copy link
Author

To add more information:

  • If I use a baudrate of 115200 I get the following error:

image

  • Drivers for the USB to UART adapter seem to be installed properly:

image

  • Tried giving rw permissions to tty/USB0 manually as well:

image

I can send more information if required.

@flynneva
Copy link
Owner

hey @Danilrivero! I haven't actually had a bno055 to test this driver in awhile - but from what I can tell the last change to the UART side of things for this driver was this commit. Could you maybe try to revert that change and see if UART works?

I really need to get around to adding some integration tests to this repo...but havent had much time unfortunately.

@Danilrivero
Copy link
Author

Hello @flynneva that seems to be the issue. Thanks for your comment!

Closing the issue due to the fix being found.

@flynneva
Copy link
Owner

@Danilrivero ok awesome. I'll go ahead and remove this change from main then, thanks for confirming that reverting that line fixed it!

@flynneva flynneva reopened this Apr 25, 2023
@flynneva
Copy link
Owner

@emilnovak see the thread above. Since you authored the problem commit here, could you give some background on why you made that change / what it fixed for you?

Maybe we are just missing an if / else somewhere

@emilnovak
Copy link
Contributor

Sorry for the late reply @flynneva, I completely missed this thread. For future reference the reasoning can be found here: #52 (comment). I may be able to look further into this problem, since I'm actively using some BNO sensors with this change via serial.

@flynneva
Copy link
Owner

@emilnovak no worries! I know how it goes - I haven't touched this repo in a long time either. I might have some time this week to put a bit of work in here as well...let's see 👍

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

No branches or pull requests

3 participants