-
Notifications
You must be signed in to change notification settings - Fork 281
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
Add camera video recorder system #316
Conversation
2dce495
to
ee796e5
Compare
Works great. Just a few comments. |
ee796e5
to
d90f78e
Compare
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.
Only nit would be: Is there a way to expose available video formats, or at least give a warning when there is an incorrect format?
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
36323b5
to
ed72b72
Compare
added check and print out error if format not supported, and also lists available formats. ed72b72 |
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.
The new test is failing on Linux and on Mac:
/var/lib/jenkins/workspace/ignition_gazebo-ci-pr_any-ubuntu_auto-amd64/ign-gazebo/test/integration/camera_video_record_system.cc:80
Value of: hasService
Actual: false
Expected: true
53: [ RUN ] CameraVideoRecorderTest.RecordVideo
53: [Wrn] [URI.cc:580] Unable to parse URI [/Users/jenkins/workspace/ignition_gazebo-ci-pr_any-homebrew-amd64/ign-gazebo/test/worlds/camera_video_record.sdf]. Ignoring.
53: objc[52149]: Class OgreConfigWindowDelegate is implemented in both /usr/local/opt/ogre1.9/lib/libOgreMain.1.9.0.dylib (0x116d86dc0) and /usr/local/opt/ogre2.1/lib/libOgreMain.2.1.0.dylib (0x1167af838). One of the two will be used. Which one is undefined.
53: dyld: lazy symbol binding failed: Symbol not found: __ZN4Ogre2v113OverlaySystemC1Ev
53: Referenced from: /usr/local/Cellar/ignition-rendering3/3.1.0/lib/libignition-rendering3-ogre2.dylib
53: Expected in: flat namespace
53:
53: dyld: Symbol not found: __ZN4Ogre2v113OverlaySystemC1Ev
53: Referenced from: /usr/local/Cellar/ignition-rendering3/3.1.0/lib/libignition-rendering3-ogre2.dylib
53: Expected in: flat namespace
53:
53/138 Test #53: INTEGRATION_camera_video_record_system ...............Child aborted***Exception: 1.18 sec
Signed-off-by: Ian Chen <[email protected]>
@osrf-jenkins run tests please |
Codecov Report
@@ Coverage Diff @@
## ign-gazebo3 #316 +/- ##
===============================================
+ Coverage 77.19% 77.24% +0.05%
===============================================
Files 197 197
Lines 10479 10479
===============================================
+ Hits 8089 8095 +6
+ Misses 2390 2384 -6
Continue to review full report at Codecov.
|
@osrf-jenkins run tests please |
latest CI builds look good. Failing tests on homebrew are unrelated. |
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.
The test failures have been fixed, so I can remove my request for changes. I haven't reviewed the PR, but the 2 approvals are enough.
Add camera video recorder system that can be attached to a camera sensor and provides a service for recording a video from the sensor's image stream.
To test, launch the
camera_video_record_dbl_pendulum.sdf
world:Call the following service to start video recording:
Call the following service to stop video recording:
The video will be saved to
test.mp4