[Enhance] Calculate and adjust FPS for video output to maintain origi… #2907
+27
−1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Match output video duration to original duration
Motivation
The current spatial-temporal action detection demo script in MMAction2 outputs a video with a fixed frame rate (default 6 FPS) without ensuring that the output video duration matches the original input video. This often leads to shorter (or longer) output videos that are temporally misaligned with the input, which is especially problematic when trying to:
This PR ensures the output video duration remains consistent with the original video by adapting either the frame sampling or the FPS accordingly.
Modification
Added logic to:
Inserted this logic before video saving with
moviepy
.The change is minimally invasive and does not affect any core inference functionality.
BC-breaking (Optional)
No, this PR is not backward compatibility breaking.
It only affects the video output format of the demo script, and improves its correctness.
Use cases (Optional)
Checklist