-
Notifications
You must be signed in to change notification settings - Fork 13
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
delay on fixture #11
Comments
Which option you prefer? 1) having state port? <module>
<name>iCub_SIM</name>
<node>localhost</node>
<ensure command="state" reply="ready">/icubSim/rpc</ensure>
</module> 2) having time delay? <module>
<name>iCub_SIM</name>
<node>localhost</node>
<wait>5.0</wait>
</module> 3) having both mechanisms? |
I implemented a simple waiting mechanism (in yarpmanager) to "ensure" that enough time is given to a module to get ready after launching it (please update to robotology/yarp@becdbbe). Using the following example, the fixture manager waits for 10s after launching the <module>
<name>iCub_SIM</name>
<parameters></parameters>
<node>localhost</node>
<ensure>
<wait>10</wait>
</ensure>
</module> Notice: The camera tests still fail (some times) on my machine even waiting for 30s before launching the test cases! @traversaro How is the behavior in Gazebo? |
👍 We should check why it takes so much to get to the correct framerate. Have you tried just to run the simulator alone and connect yarp ping --rate to one of the cameras? |
I have tested the icubSim cameras rate using our <?xml version="1.0" encoding="UTF-8"?>
<suit name="Basic Tests Suite">
<description>Testing robot's basic features</description>
<environment>--robotname icubSim</environment>
<fixture param="--fixture icubsim-fixture.xml"> yarpmanager </fixture>
<!-- Port frequency (camera) -->
<test type="dll" param="--from cam_stream.ini"> PortsFrequency </test>
<!-- Camera -->
<test type="dll" param="--from camera_right.ini"> CameraTest </test>
<test type="dll" param="--from camera_left.ini"> CameraTest </test>
</suit>
|
In theory this kind of wait should not be necessary on Gazebo, however I obtained strange results when running the Gazebo tests on Travis (see robotology/gazebo-yarp-plugins#194 ). I could add a waiting time to see if the problem persist. |
iCubSim takes some time to stabilize framerate: first execution of camera test fails because of this. Maybe we should add the possibilty to add a delay after the fixture:
The text was updated successfully, but these errors were encountered: