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.

Sign up for GitHub

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

Jump to bottom

ModuleNotFoundError: No module named 'tf2_sensor_msgs' #471

Closed
YoushaaMurhij opened this issue Oct 22, 2021 · 8 comments
Closed

ModuleNotFoundError: No module named 'tf2_sensor_msgs' #471

YoushaaMurhij opened this issue Oct 22, 2021 · 8 comments
Labels
backlog

Comments

@YoushaaMurhij
Copy link

Is there any plan to include tf2_sensor_msgs?
@clalancette
Copy link
Contributor

Eventually, yes. But it is blocked by ros2/orocos_kinematics_dynamics#19

@Flova
Copy link
Contributor

Flova commented Mar 4, 2022

Same with tf2_geometry_msgs.

@Flova Flova mentioned this issue Mar 5, 2022
Merged
@ozangungor12
Copy link

ozangungor12 commented Mar 18, 2022 •
edited
Loading

Sorry for the basic question, but I find it quite challenging to understand what is ported and what is not from ROS1 to ROS2.
Since I already see the tf2_geometry_msgs in this repo, I thought it was already available.
Also sudo apt-get install ros-foxy-tf2-geometry-msgs says that the package is already the newest version.
What is the easy way of understanding if a package in a repo is already ported or not? Is it the missing setup.py file?

@Flova
Copy link
Contributor

Flova commented Mar 18, 2022

The python part of the tf2_geometry_msgs was excluded by deleting its part from the CMakeLists.txt file, which disables the build of this part of the package. It is not very obvious that this part of the code is not being build. I agree with you. The code is already ported by deactivated due to an upstream dependency issue. This should be resolved by dropping the dependency as proposed in this PR #509.

aprotyas added a commit that referenced this issue Apr 27, 2022
@Flova @aprotyas
Port point cloud transformation to numpy (#507)
6242f01 
Implements the point cloud transformation in NumPy. This speeds
up the transformation operation by 20000% for large point clouds
(excluding the serialization and deserialization). The code needs
some further testing but should work as it is. I also try to improve
the performance of the serialization and deserialization as seen in
ros2/common_interfaces#175.

This adds NumPy as a dependency and drops the dependency on
PyKDL which should solve #471.

The API is also improved by adding proper interface descriptions.

Signed-off-by: Florian Vahl <[email protected]>
Co-authored-by: Abrar Rahman Protyasha <[email protected]>
@HenryPCardoso
Copy link

just use the command:
sudo apt-get install ros-foxy-tf2-sensor-msgs
It worked fine for me

@Flova
Copy link
Contributor

Flova commented Oct 19, 2023

Imo. this issue can be closed as #509 is merged for a while now.

@Flova
Copy link
Contributor

Flova commented Oct 19, 2023

just use the command: sudo apt-get install ros-foxy-tf2-sensor-msgs It worked fine for me

It could not be build due to a missing dependency prior to #509 iirc.. It was therefore no binary was released.

Check your package.xml dependency definitions and run rosdep again if you need to install it explicitly like you did.

@clalancette
Copy link
Contributor

Yes, this was long ago enabled, and is currently available in all of our supported distributions. I'm going to close this one out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backlog
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@clalancette @Flova @YoushaaMurhij @ozangungor12 @HenryPCardoso