Skip to content

Conversation

@JAP3TH
Copy link

@JAP3TH JAP3TH commented Oct 24, 2022

To build carla_ros_bridge in ROS2, additionally the tf2_eigen library is required.


This change is Reviewable

@kamibo
Copy link

kamibo commented Oct 27, 2022

+1

1 similar comment
@zeidk
Copy link

zeidk commented Mar 22, 2023

+1

@johannesWen
Copy link

+1

Copy link

@oliver-schumann oliver-schumann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @JAP3TH)

Copy link

@oliver-schumann oliver-schumann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes are critical. Otherwise the build with newer ROS builds as Humble and Iron fails. We need to support these dists, especially if so little changes are necessary.

Copy link

@DaniCarias DaniCarias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me in Ubuntu 22.04 ROS Humble

@AxoyTO
Copy link

AxoyTO commented May 15, 2025

+1

@TommasoLabieni
Copy link

Can we please have this PR merged? On ROS2 Humble there's still the include error for tf2_eigen/tf2_eigen.h as already mentioned on issue #737.

Thanks!

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

Successfully merging this pull request may close these issues.

10 participants