[FR] Let fiftyone.core.video.make_frames_dataset take kwargs for ffmpeg.

[bhoo-git]

Proposal Summary

The fiftyone.core.video.make_frames_dataset function uses the fiftyone.utils.video.sample_videos function under the hood. The fiftyone.utils.video.sample_videos function can take keyword arguments for eta.core.video.FFmpeg(**kwargs). It would be nice to have fiftyone.core.video.make_frames_dataset also be able to take keyword arguments and have it feed the FFmpeg args downstream.

Motivation

• What is the use case for this feature? Making the image datasets from videos more streamlined.
• Why is this use case valuable to support for FiftyOne users in general? It gives more flexibility to the user when creating image dataset(s) from frames of video(s) within the existing function use case.
• Why is this use case valuable to support for your project(s) or organization? I expect to get raw videos and want to store the frames in a FiftyOneDataset format as a part of my workflow.
• Why is it currently difficult to achieve this use case? Without the keyword arguments, the frames are automatically stored as a .jpg extensions and sometimes the image quality suffers from the compression.

What areas of FiftyOne does this feature affect?

• App: FiftyOne application
• Core: Core fiftyone Python library
• Server: FiftyOne server

Details

It would be a pretty simple change in my opinion, would just have to let make_frames_dataset take **kwargs and have it fed into the fouv.sample_videos.

Willingness to contribute

The FiftyOne Community welcomes contributions! Would you or another member of your organization be willing to contribute an implementation of this feature?

• Yes. I can contribute this feature independently
• Yes. I would be willing to contribute this feature with guidance from the FiftyOne community
• No. I cannot contribute this feature at this time