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

os_sensor node error when trying to publish imu/lidar_packets (ros2-foxy) #212

Closed
tseco15 opened this issue Sep 12, 2023 · 5 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@tseco15
Copy link

tseco15 commented Sep 12, 2023

os_sensor node error when trying to publish imu/lidar_packets (ros2-foxy)
We are working with ros2-foxy branch and we need lidar_packets topic.
When we launch the node, we get the following warning:

[os_sensor-1] [WARN] [1694514333.922770617] [LifecyclePublisher]: Trying to publish message on the topic '/ouster/imu_packets', but the publisher is not activated
[os_sensor-1] [WARN] [1694514333.923958652] [LifecyclePublisher]: Trying to publish message on the topic '/ouster/lidar_packets', but the publisher is not activated

but os_sensor is configured and activated.

To Reproduce
In order to reproduce the bug, we launch the os_sensor by two different ways:

  1. ros2 launch ouster_ros sensor.independent.launch.xml sensor_hostname:=192.168.1.60
    Once the connection with the ouster lidar is established, the warnings arise. The /ouster/points are not published either.

  2. ros2 launch ouster_ros sensor.composite.launch.py
    We change the value of sensor_hostname in os_sensor_cloud_image_params.yaml config file (default one), but the config file is not loaded correctly because an error arises:
    [ouster.os_sensor]: Must specify a sensor hostname
    Anyway, we set the sensor_hostname value manually in the source code, and after launching the previous launch file, the same message about the activation is shown.

Platform :

  • Ouster Sensor: OS-1-32-U1
  • Ouster Firmware Version: v2.4.0
  • ROS 2 foxy
  • Ubuntu 20.04.5 LTS
  • Jetson Xavier
@tseco15 tseco15 added the bug Something isn't working label Sep 12, 2023
@Samahu
Copy link
Contributor

Samahu commented Sep 12, 2023

First time I have seen a complain about the topics are not activated, are using the latest commit.

ros2 launch ouster_ros sensor.composite.launch.py
We change the value of sensor_hostname in os_sensor_cloud_image_params.yaml config file (default one), but the config file is not loaded correctly because an error arises:
[ouster.os_sensor]: Must specify a sensor hostname
Anyway, we set the sensor_hostname value manually in the source code, and after launching the previous launch file, the same message about the activation is shown.

This error suggest that the file is not properly loaded or maybe the namespace/nodename combination does not match. Since you are using the default file, the path shouldn't have be the issue. So I am wondering if changed the node name?

@tseco15
Copy link
Author

tseco15 commented Sep 13, 2023

First time I have seen a complain about the topics are not activated, are using the latest commit.

Thanks for your reply.
Regarding your comments:

  • We are using the latest commit: commit 043d1a9
  • The combination namespace/nodename is the default one: /ouster/os_sensor. The configuration file also refers to this node
  • This node is active, the connection is established, but the error about trying to publish imu/laser_packets is raised
  • If we launch only os_driver node (using driver.launch.py), we get all the topics published by this node: points, imu and so on, but this node does not publish lidar_packets topic we need. For this reason, we tried with os_sensor node that publishes this topic
  • We also tried exactly the same but with ROS Humble and ros2 branch and everything works properly, but unfortunately, our hardware platform only works with Ubuntu 20 (Jetson Xavier)

Any suggestions?

@Samahu
Copy link
Contributor

Samahu commented Sep 13, 2023

@tseco15 This was a code issue that was inadvertently introduced by a previous merge. Please update to the latest commit for the ros2-foxy branch {package-version: 0.10.4} and this issue should be gone.

@Samahu Samahu closed this as completed Sep 13, 2023
@tseco15
Copy link
Author

tseco15 commented Sep 14, 2023

@tseco15 This was a code issue that was inadvertently introduced by a previous merge. Please update to the latest commit for the ros2-foxy branch {package-version: 0.10.4} and this issue should be gone.

Thank you very much! It works properly now.
The issue regarding the parameters file loading with sensor.composite.launch.py is not still solved, but it is not critical for us, we can work with any other launch file.
Thanks again!

@Samahu
Copy link
Contributor

Samahu commented Sep 18, 2023

With regard to sensor.composite.launch.py I looked briefly into the issue and it seems related to node namespaces. Somehow Foxy does not work well when LifecycleNode is used along with composition and namespaces are used 🤷‍♂️ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants