Fix buildfarm tests again (increase test timeout)

locusrobotics / robot_navigation

Spiritual successor to ros-planning/navigation.

450 stars 148 forks source link

Fix buildfarm tests again (increase test timeout) #30

Closed mintar closed 5 years ago

mintar commented 5 years ago

The current timeout of 1080 s (18 minutes) is too low for completing the tests. On my laptop, the total required time is 1222 s (20 min 22 s). On the buildfarm, it's also over 18 minutes.

These are the timings on my four-year-old development laptop (Intel(R) Core(TM) i7-4710MQ CPU @ 2.50GHz, 8 GB RAM):

Test	Duration
AStarVon	302 s
AStarGrid	297 s
AStarGradient	328 s
DijkstraVon	89 s
DijkstraGrid	92 s
DijkstraGradient	114 s

mintar commented 5 years ago

Please don't merge yet, I'm waiting for the buildfarm tests to complete.

mintar commented 5 years ago

For the record, the buildfarm took 19 min 41 sec for this test, so it's about the same speed as my laptop. According to your launch file comment your laptop finishes in 6 minutes - I should get a new one! :)

I'm good to merge as well.