search

flynneva / bno055

ROS2 driver for Bosch BNO055 using UART or I2C
https://flynneva.github.io/bno055/
BSD 3-Clause "New" or "Revised" License
83 stars 50 forks source link

Communication error using UART to USB #63

Open Danilrivero opened 1 year ago

Danilrivero commented 1 year ago

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[bot] commented 1 year ago

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

Danilrivero commented 1 year ago

To add more information:

image

image

image

I can send more information if required.

flynneva commented 1 year ago

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 commented 1 year ago

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

Closing the issue due to the fix being found.

flynneva commented 1 year ago

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

flynneva commented 1 year ago

@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 commented 9 months ago

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 commented 9 months ago

@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 👍