Updates to camera video record from subt

[nkoenig]

Signed-off-by: Nate Koenig [email protected]

🎉 New feature

Summary

Backports camera video recording updates from Dome that enables support for simtime recording. I've also added in a marker manager so that markers can be inserted into the scene used by the camera recorder.

This code was used to produce robot trajectory videos during subt.

Test it

The existing camera_video_record_dbl_pendulum.sdf should still work.

Checklist

• Signed all commits for DCO
• Added tests
• Added example and/or tutorial
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

[codecov]

Codecov Report

Merging #1117 (2e39b5e) into ign-gazebo3 (9d0dfc3) will not change coverage.
The diff coverage is n/a.

❗ Current head 2e39b5e differs from pull request most recent head 13225b3. Consider uploading reports for the commit 13225b3 to get more accurate results

@@             Coverage Diff              @@
##           ign-gazebo3    #1117   +/-   ##
============================================
  Coverage        77.70%   77.70%           
============================================
  Files              222      222           
  Lines            12716    12716           
============================================
  Hits              9881     9881           
  Misses            2835     2835           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d0dfc3...13225b3. Read the comment docs.

[mjcarroll]

One comment question.

I think there will be some issues porting this across to fortress because the marker manager has been moved into ign-gui.

[nkoenig]

I think there will be some issues porting this across to fortress because the marker manager has been moved into ign-gui.

Yeah, I'll have to take a look at this when I forward port this PR.