ros2 / rmw_zenoh

RMW for ROS 2 using Zenoh as the middleware
Apache License 2.0
144 stars 29 forks source link

Updated zenoh-c to latest commit #164

Closed imstevenpmwork closed 2 months ago

imstevenpmwork commented 2 months ago

Hi @Yadunund! We're submitting this PR to ensure we use the latest zenoh-c commit before proceeding with our planned testing

clalancette commented 2 months ago

I think I'm a little confused by this PR. As far as I understood, if we bump to the latest zenoh-c, then some of our tests stop working. We are still debugging that. Am I misunderstanding, or did we solve that particular issue?

imstevenpmwork commented 2 months ago

Hello @clalancette! This pull request targets the branch to evaluate rmw with Tokio zenoh-c. Although the current hash already corresponds to Tokio zenoh-c, it's a few weeks outdated. Hence, we believe it's prudent to update this branch to the latest Tokio zenoh-c version and rerun the tests to assess any potential improvements or regressions.