Added run to time feature #478
Conversation
Signed-off-by: Nate Koenig <nate@openrobotics.org>
There was a problem hiding this comment.
Could you explain the use case for this feature that couldn't be handled by the user through the multi_step field?
I also have a concern on bumping the major version of ign-msgs and ign-transport for Edifice just for this feature. Are there more changes planned to those libraries to help justify the new versions?
chapulina
added
the
needs upstream release
In order to step to a simulation time, I would need to know the step size. This can be accomplished, but it's not user friendly especially from the command line. |
|
Since we're bumping the |
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Codecov Report
@@ Coverage Diff @@
## main #478 +/- ##
==========================================
- Coverage 77.38% 77.35% -0.03%
==========================================
Files 213 213
Lines 11950 11970 +20
==========================================
+ Hits 9247 9259 +12
- Misses 2703 2711 +8
Continue to review full report at Codecov.
|
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
chapulina
removed
the
needs upstream release
This should be ok, the CI errors are coming from other things. |
Add the ability to set a run-to-time. This will allow a user to set the simulation time that simulation should run to and then pause.
Requires gazebosim/gz-msgs#108.
Signed-off-by: Nate Koenig nate@openrobotics.org