Open LanderU opened 5 years ago
@LanderU thanks for the report. Do you happen to have a regression test for this issue? If not, it's ok, I'll write one, it's just for not duplicating work.
Do you happen to have a regression test for this issue?
No, we don't have any test. We just compile against dashing and run the performance_test as we did with crystal.
No, we don't have any test. We just compile against dashing and run the performance_test as we did with crystal.
No problem, I'll add one to have it run by the CI.
Hey, I have just reproduce this issue and I have the same issue. Any update?
@abilbaotm no updates, I just got back from traveling. I'll be working on a test soon.
@esteve: Do you need help with this?
@deeplearningrobotics could you have a look at https://github.com/ApexAI/performance_test/pull/68 and figure out how to get launch_testing
running for performance_test
? I'd like the CI to show the error and ensure we don't have a regression once it's all fixed.
I'm currently focused on Autoware and don't have time for performance_test
.
Thanks.
@pbaughman could you help @esteve here write the integration test?
@MiguelCompany you would know why we get the Unknown QoS Liveliness policy
, no?
@MiguelCompany you would know why we get the
Unknown QoS Liveliness policy
, no?
I think that, as @LanderU pointed out, the issue may be in https://github.com/ApexAI/performance_test/blob/master/performance_test/src/communication_abstractions/ros2_callback_communicator.hpp#L46
As there new fields were added on dashing for that struct, they may get randomly initialized. I think the issue may be solved by changing that line to rmw_qos_profile_t ros_qos = rmw_qos_profile_default;
@dejanpan are you talking about PR #68? The issue is almost certainly that there are no 'active' tests that run. The launch description is going to be shut down as soon as its launched because no tests are set to run concurrently with the launched processes - possibly before either of the processes under test finish starting up.
A good fix would be to add an 'active' test that uses one of the 'wait' assertions to make sure whatever is supposed to happen happens. Maybe some stdout, or monitor a ROS topic or wait for a process to terminate or something else.
Hello!
We're facing some issues after the update of the code for dashing. We're not able to run the code using the instructions of the README.md:
Error log:
We think the error is here:
Looking as example of demos:
Let me know if this make sense for you, or we're doing something wrong.
Thanks in advance!