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

structure_needs_at_least_one_member Verbosity with std_msgs/msg/Empty #297

Closed
IanTheEngineer opened this issue Jul 16, 2019 · 1 comment · Fixed by ros2/rosidl_python#73
Assignees
Labels
enhancement New feature or request

Comments

@IanTheEngineer
Copy link

Bug report

When dealing with std_msgs.msg.Empty messages, there's a repeated field output from the ros2cli: structure_needs_at_least_one_member=0. This appears on both publisher and subscriber sides.

Required Info:

  • Operating System:
    • Ubuntu 18.04
  • Installation type:
    • ROS Buildfarm Debian binaries
  • Version:
    • ROS Dashing
  • DDS implementation:
    • Fast-RTPS
  • Client library:
    • rclpy

Steps to reproduce issue:

Terminal 1

$ ros2 topic pub /my_empty_topic std_msgs/msg/Empty
publisher: beginning loop
publishing #1: std_msgs.msg.Empty(structure_needs_at_least_one_member=0)

publishing #2: std_msgs.msg.Empty(structure_needs_at_least_one_member=0)

publishing #3: std_msgs.msg.Empty(structure_needs_at_least_one_member=0)

publishing #4: std_msgs.msg.Empty(structure_needs_at_least_one_member=0)

publishing #5: std_msgs.msg.Empty(structure_needs_at_least_one_member=0)

Terminal 2

$ ros2 topic echo /my_empty_topic
structure_needs_at_least_one_member: 0
---
structure_needs_at_least_one_member: 0
---
structure_needs_at_least_one_member: 0
---
structure_needs_at_least_one_member: 0
---
structure_needs_at_least_one_member: 0

Expected behavior

The same as ROS1, just output the three dashes:

$ rostopic echo /my_empty_topic

---

---

Actual behavior

Extra structure_needs_at_least_one_member=0 per publish

@IanTheEngineer
Copy link
Author

@dirk-thomas well done on the swift fix 👍

esteve pushed a commit to esteve/ros2cli that referenced this issue Dec 16, 2022
* Add Parameter substitution

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Fix bug of set_parameter.py

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Add a test for SubstitutionFailure

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants