Fix minor benchmark compile & runtime errors due to api changes.

RobotWebTools / rclnodejs

Node.js version of ROS 2.0 client

https://docs.ros.org/en/humble/Concepts/Basic/About-Client-Libraries.html?highlight=rclnodejs#community-maintained

Apache License 2.0

316 stars 70 forks source link

Fix minor benchmark compile & runtime errors due to api changes. #745

Closed wayneparrott closed 3 years ago

wayneparrott commented 3 years ago

Commit Summary

CmakeList.txt

added example_interfaces to find_package()

publisher-stress-test.cpp

added qos argument when creating publisher
removed std::make_shared when creating the test message to resolve compilation error.

subscription-stress-test.cpp

added missing ';'

publisher-stress-test.py

added required QoS argument when creating publisher

subscription-stress-test.py

added required QoS argument when creating subscription

Fix #739

coveralls commented 3 years ago

Coverage remained the same at 91.289% when pulling 66679f6bb968c5497005c454ddc2e30c3b4ac32b on wayneparrott:benchmark_739 into 04d9539b851b860d4cdb05de631326da45ca94ae on RobotWebTools:develop.