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

Runtime error message when publishing message (ros2 rolling / Iron ) #100

Closed
felixf4xu opened this issue Sep 20, 2023 · 1 comment
Closed
Assignees
Labels
enhancement New feature or request rolling Related to ROS 2 Rolling

Comments

@felixf4xu
Copy link

Hi,

After the fix of #94 rmw_iceoryx_cpp can be compiled on Windows. Then I did some test and find an issue.

steps:

  1. start iox-roudi.exe

  2. set RMW_IMPLEMENTATION=rmw_iceoryx_cpp and then run ros2 run demo_nodes_cpp talker

then I have the following error messages:

'failed to resolve symbol 'rmw_get_gid_for_client' in shared library 'c:\ros\rolling\install\bin\rmw_iceoryx_cpp.dll', at C:\ros\rolling\src\ros2\rmw_implementation\rmw_implementation\src\functions.cpp:154'

'failed to resolve symbol 'rmw_take_dynamic_message' in shared library 'c:\ros\rolling\install\bin\rmw_iceoryx_cpp.dll', at C:\ros\rolling\src\ros2\rmw_implementation\rmw_implementation\src\functions.cpp:154'

'failed to resolve symbol 'rmw_take_dynamic_message_with_info' in shared library 'c:\ros\rolling\install\bin\rmw_iceoryx_cpp.dll', at C:\ros\rolling\src\ros2\rmw_implementation\rmw_implementation\src\functions.cpp:154'

'failed to resolve symbol 'rmw_serialization_support_init' in shared library 'c:\ros\rolling\install\bin\rmw_iceoryx_cpp.dll', at C:\ros\rolling\src\ros2\rmw_implementation\rmw_implementation\src\functions.cpp:154'

As for rmw_get_gid_for_client, it's added in ROS2 Iron (search "rmw_get_gid_for_client" at https://docs.ros.org/en/iron/Releases/Iron-Irwini-Complete-Changelog.html), and I'm testing on ros2 rolling, so it might be fine in humble but not in Iron when it's planned to be supported in #99

BTW, to my surprise, the talker/listener demo still work great despite of the error message. 💯

@felixf4xu felixf4xu changed the title Runtime error message when publishing message on Windows Runtime error message when publishing message (ros2 rolling) Sep 20, 2023
@felixf4xu felixf4xu changed the title Runtime error message when publishing message (ros2 rolling) Runtime error message when publishing message (ros2 rolling / Iron ) Sep 20, 2023
@mossmaurice mossmaurice added rolling Related to ROS 2 Rolling enhancement New feature or request labels Sep 20, 2023
@mossmaurice
Copy link
Collaborator

@felixf4xu Thanks for trying out the changes and for opening the issue.

After the fix of #94 rmw_iceoryx_cpp can be compiled on Windows
BTW, to my surprise, the talker/listener demo still work great despite of the error message. 💯

Great! 🎉

then I have the following error messages:

I saw the same runtime errors when running rolling. I think all four methods were introduced with iron.

Hence, I added the following as a todo item to #99

  1. rmw_get_gid_for_client
  2. rmw_take_dynamic_message
  3. rmw_take_dynamic_message_with_info
  4. rmw_serialization_support_init

I'm testing on ros2 rolling, so it might be fine in humble

If you have some time it would be awesome if you could also test humble.

@mossmaurice mossmaurice self-assigned this Oct 24, 2023
mossmaurice added a commit to ApexAI/rmw_iceoryx that referenced this issue Oct 25, 2023
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
mossmaurice added a commit to ApexAI/rmw_iceoryx that referenced this issue Oct 25, 2023
Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
mossmaurice added a commit that referenced this issue Nov 13, 2023
…unt_*` functions (#102)

* Implement new rmw functions needed for iron (#99)

Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>

* Implement stubs for new 'rmw_count_*' functions (#100)

Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>

---------

Signed-off-by: Simon Hoinkis <simon.hoinkis@apex.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request rolling Related to ROS 2 Rolling
Projects
None yet
Development

No branches or pull requests

2 participants