Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Switch to the c++ wrapper for playback #72

Closed
RoseFlunder opened this issue Sep 10, 2019 · 0 comments
Closed

Switch to the c++ wrapper for playback #72

RoseFlunder opened this issue Sep 10, 2019 · 0 comments
Labels
enhancement New feature or request triage needed The Issue still needs to be reviewed by the Azure Kinect ROS Driver Team

Comments

@RoseFlunder
Copy link
Contributor

RoseFlunder commented Sep 10, 2019 •
edited

Is your feature request related to a problem? Please describe.
To resolve deprecation warnings and also for a less verbose code we should switch to the new c++ wrapper for the playback-api that came with sensor sdk version 1.2.0.

Describe the solution you'd like
Use k4arecord/playback.hpp instead of k4arecord/playback.h
@RoseFlunder RoseFlunder added enhancement New feature or request triage needed The Issue still needs to be reviewed by the Azure Kinect ROS Driver Team labels Sep 10, 2019
@RoseFlunder RoseFlunder mentioned this issue Sep 10, 2019
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request triage needed The Issue still needs to be reviewed by the Azure Kinect ROS Driver Team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@skalldri @RoseFlunder