search

microsoft / Azure-Kinect-Sensor-SDK

A cross platform (Linux and Windows) user mode SDK to read data from your Azure Kinect device.
https://Azure.com/Kinect
MIT License
1.49k stars 619 forks source link

k4arecorder capability to record only frames with synchronized depth/color frames #557

Open mffigueroa opened 5 years ago

mffigueroa commented 5 years ago

Is your feature request related to a problem? Please describe. In k4arecorder, there is currently no command-line option to record only frames with both depth and color frames received. I.e. to set the synchronized_images_only option in k4a_device_configuration_t.

Describe the solution you'd like Add a --synchronized_images_only command-line option to set this option when recording a clip.

wes-b commented 5 years ago

Why would the recorder need this? My thinking is that the recorder should record what it gets. Playback APIs should have the functionality to only deliver synchronized frames.

Get Outlook for Androidhttps://aka.ms/ghei36

From: Mikael Figueroa notifications@github.com Sent: Monday, July 29, 2019 12:07:49 PM To: microsoft/Azure-Kinect-Sensor-SDK Azure-Kinect-Sensor-SDK@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: [microsoft/Azure-Kinect-Sensor-SDK] k4arecorder capability to record only frames with synchronized depth/color frames (#557)

Is your feature request related to a problem? Please describe. In k4arecorder, there is currently no command-line option to record only frames with both depth and color frames received. I.e. to set the synchronized_images_only option in k4a_device_configuration_t.

Describe the solution you'd like Add a --synchronized_images_only command-line option to set this option when recording a clip.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAzure-Kinect-Sensor-SDK%2Fissues%2F557%3Femail_source%3Dnotifications%26email_token%3DACVJTJN3R4RGTZF6U5NV3V3QB45YLA5CNFSM4IHVK762YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HCD3KGQ&data=02%7C01%7CWes.Barcalow%40microsoft.com%7C95b0015e1c3d4cbbb59508d714580d15%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637000240723445764&sdata=xl%2FpV8PGPTTMKRqjbylB%2BqcQFG3x%2BV6O9Zd8ySHmEkU%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACVJTJOMBABHJEX6T2CPXSTQB45YLANCNFSM4IHVK76Q&data=02%7C01%7CWes.Barcalow%40microsoft.com%7C95b0015e1c3d4cbbb59508d714580d15%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637000240723445764&sdata=IQ7gJGrd6gGAz7f55ng%2F57YrvcFyPofikIZwnGCZ7K4%3D&reserved=0.

cdedmonds commented 5 years ago

I think the problem is that the playback API does not have this functionality currently.

wes-b commented 5 years ago

Gotcha, @xthexder how about we add k4a_playback_set_sychronized_images_only()?

xthexder commented 5 years ago

I think this was originally a feature, but was cut due to confusion over what the behavior should be. If a recording only has color enabled, or only depth/ir, would the playback API drop every capture? Or should we still return the single track? I'm not sure what the current behavior of synchronized_images_only is when the camera is only running a single stream.