search

rock-gazebo / simulation-rock_gazebo

Main package that deals with the integration of Rock and Gazebo
Other
0 stars 1 forks source link

completely remove the default rate test #33

Closed doudou closed 4 years ago

doudou commented 4 years ago

The "explicit rate" test has not failed (yet), and let's hope it will stay that way. The lesson here is that if you want some rate, make sure that (1) Gazebo runs faster than the rate you want and (2) you explicitly set it.

Event at 10Hz, the explicit rate test was working at least more often than the default. It seems that something's wrong on the overall logic.

doudou commented 4 years ago

GPSTask test fails:

1: F..........................................................
1: 
1: Finished in 140.626875s, 0.8604 runs/s, 2.0195 assertions/s.
1: 
1:   1) Failure:
1: rock_gazebo::GPSTask::common sensor behavior for rock_gazebo::GPSTask#test_0004_outputs one sample per cycle by default [/simulation/rock_gazebo/bindings/ruby/test/helpers.rb:218]:
1: Expected 9..11 to include 8.
1: 
1: 121 runs, 284 assertions, 1 failures, 0 errors, 0 skips
1/1 Test #1: test-rock_gazebo-ruby ............***Failed  141.78 sec

0% tests passed, 1 tests failed out of 1

Total Test time (real) = 141.79 sec

The following tests FAILED:
          1 - test-rock_gazebo-ruby (Failed)
Errors while running CTest

I'm going to cry ...

doudou commented 4 years ago

I just removed the whole thing.