Closed brimoor closed 1 month ago
The recent changes involve modifying the string formatting for file path generation in the _do_download_media
function within the fiftyone/utils/cvat.py
file. The dot separating the chunk ID from the file extension has been removed, changing the file naming convention. This adjustment may influence media concatenation processes and file retrieval mechanisms.
Files | Change Summary |
---|---|
fiftyone/utils/cvat.py | Modified string formatting for chunk_path in _do_download_media by removing the dot between chunk_id and ext . |
Objective | Addressed | Explanation |
---|---|---|
Fix video concatenation failure (#4627) | ❌ | The change in file naming format may disrupt file retrieval needed for concatenation. |
In the land of code, where rabbits hop,
I changed a dot, and now it won't stop!
Files will dance in paths anew,
Concat those videos, a joyous view!
With every fix, we bound and play,
Hopping along the coding way! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Resolves https://github.com/voxel51/fiftyone/issues/4627
As pointed out by @ChicoQi, the current implementation of the CVAT chunked video download chooses temp filenames with double
..
in them:and apparently some versions of
ffmpeg
don't like that.With this patch, the temp filenames will have single
.
:Summary by CodeRabbit