-
Notifications
You must be signed in to change notification settings - Fork 277
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
Support locked entities, and headless video recording using sim time #862
Conversation
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-gazebo4 #862 +/- ##
===============================================
+ Coverage 67.10% 67.12% +0.01%
===============================================
Files 243 243
Lines 18212 18250 +38
===============================================
+ Hits 12222 12251 +29
- Misses 5990 5999 +9
Continue to review full report at Codecov.
|
The sim time video recording feature is working for me. I also tried running faster than real time and that's working as well and produced the correct video length. (RTF just drops a little when video recording is on). another nice video recorder param that we could port over later from Scene3D is so users can record high quality videos. |
|
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with Green CI (minus tests we know are flaky)
Signed-off-by: Nate Koenig [email protected]
🎉 New feature
Related PR: gazebosim/gz-sensors#137
Summary
The goal of this PR is to support generating videos from log files using a headless simulation instance. In order to accomplish this, I need to spawn a camera into simulation and keep that camera around during log seeks. This lead to the need for
locked
entities within the ECM.Recording a video using simulation time is required when you want a video to playback at real-time speeds. I copied the sim time video recording code from the Scene3d plugin.
Test it
There are ECM tests for the locking feature.
The easiest way to test the sim time video recording is to use the
camera_video_record_dbl_pendulum.sdf
example world file, and add<use_sim_time>true</use_sim_time>
to the plugin. Depending on your machine, you may have to adjust the physics step to make sure simulation is running slower than real-time. Record a video, and note the simulation time duration for the video. The resulting video should have matching duration, which can be inspected using:mplayer -identify VIDEO.MP4 2>/dev/null | grep ID_LENGTH
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge