Leak_Technologies/VideoTools
Leak_Technologies/VideoTools
2
1
Fork 1
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

Fix snippet duration: revert to simple, reliable FFmpeg approach

Browse Source 
Reverts the problematic -accurate_seek and -to flags that caused wildly incorrect durations (9:40 instead of 10s). Returns to the standard, reliable FFmpeg pattern for stream copy:

ffmpeg -ss START -i input -t DURATION -c copy output

This places -ss before -i for fast keyframe seeking and uses -t for duration (not -to which is an absolute timestamp causing incorrect extraction). Should now correctly extract the configured snippet length centered on video midpoint.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
Stu Leak 2025-12-16 23:36:04 -05:00
parent 1a04cab1d6
commit 9fbc791e57
1 changed files with 8 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
main.go
View File

@ -3366,20 +3366,17 @@ func (s *appState) executeSnippetJob(ctx context.Context, job *queue.Job, progre
if useSourceFormat {
// Source format mode: Use stream copy for clean extraction
// Note: This uses keyframe cutting, so duration may not be frame-perfect
// Calculate end time for more accurate cutting
endTime := center + float64(snippetLength)
// Simple, reliable approach: -ss before -i for fast seek, -t for duration
args = []string{
"-y",
"-ss", start, // Seek to start position (before -i for fast keyframe seek)
"-i", inputPath,
"-t", fmt.Sprintf("%d", snippetLength), // Duration to extract
"-c", "copy", // Stream copy - no re-encoding
"-map", "0", // Include all streams
"-avoid_negative_ts", "make_zero", // Fix timestamp issues
"-y", // Overwrite output
"-hide_banner",
"-loglevel", "error",
"-accurate_seek", // Seek accurately to keyframes
"-ss", start,
"-i", inputPath,
"-to", fmt.Sprintf("%.2f", endTime), // Use -to instead of -t for better accuracy
"-c", "copy", // Stream copy - no re-encoding
"-map", "0", // Include all streams
"-avoid_negative_ts", "make_zero", // Fix timestamp issues
outputPath,
}
} else {