perf: Optimise FFmpeg args and show elapsed export time (#694)#1040
Conversation
|
Someone is attempting to deploy a commit to the magic-peach1's projects Team on Vercel. A member of the Team first needs to authorize it. |
✅ PR Format Check Passed — @Abhi190702Basic format checks passed. A maintainer will review your code changes. This does not mean the PR is approved — it just means the format is correct. |
👋 Thanks for your PR, @Abhi190702!Welcome to Reframe — a browser-based video editor built for everyone 🎬 What happens next
Quick checklist
Useful links
Happy coding! 🎉 |
|
@magicPeach , adding recording soon !! |
|
@magic-peach , happy that You might look into this |
|
Merged! This PR delivers meaningful performance improvements (ultrafast preset, VP9 realtime mode, skipping neutral eq filter) plus a helpful elapsed export timer in the UI. Well-tested with the regression test for neutral adjustments. All CI checks passed. Excellent work @Abhi190702! |
|
@magic-peach , Thanks , Looking forward for More contributions and support |
New Reframe - (30 sec clip showing , relatively fast export)
Reframe-New.online-video-cutter.com.mp4
Old Reframe - (30 sec clip showing , slow export)
Reframe-Old.online-video-cutter.com.mp4
Summary
Resolves #694
Optimises FFmpeg export settings so browser-based exports complete much faster, and adds visible elapsed export timing during and after export.
Changes Made
-preset mediumto-preset ultrafast-cpu-used 4 -deadline realtimeeqfilter when brightness/contrast/saturation are neutral0, contrast1, saturation10:42 elapsedExported in 1 min 23 secuseVideoEditoreqCOOP/COEP
vercel.jsonalready contains:Cross-Origin-Opener-Policy: same-originCross-Origin-Embedder-Policy: require-corpNo hosting-header changes were needed in this PR.
Testing
bun run lint✅bunx tsc --noEmit✅bun run test -- --run✅bun run build✅Note:
bun run lintandbun run buildstill print the existingThumbnailStrip.tsxhook warning frommain, but both commands exit successfully. The build required elevated filesystem access locally because Next.js hit Windows sandboxreadlinkpermissions.Screen Recordings
Required before merge:
main