-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: video rendering issue in windows #372
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
screenpipe-server/src/video.rs
Outdated
args.extend_from_slice(&["-vcodec", "libx264", "-preset", "ultrafast", "-crf", "23"]); | ||
} | ||
// Use libx264 for all platforms | ||
args.extend_from_slice(&["-vcodec", "libx264", "-preset", "ultrafast", "-crf", "23", ]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how did you install ffmpeg? @tribhuwan-kumar
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ffmpeg is available in window's official package manager winget
also for development purpose I installed ffmpeg from vcpkg.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so this will crash
you should use the ffmpeg version from the pre_build.js
which is packaged in the release of the app (or change this to use a "better" version of ffmpeg) the if i did is because the ffmpeg does not support the arg given here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#194 would probably resolve all the problems i guess
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so this will crash
you should use the ffmpeg version from the
pre_build.js
which is packaged in the release of the app (or change this to use a "better" version of ffmpeg) the if i did is because the ffmpeg does not support the arg given here
Okay, I'm uninstalling the downloaded version of ffmpeg from winget & vcpkg and try to rebuild it with pre_build.js
version of ffmpeg.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you should use the ffmpeg version from the
pre_build.js
I did some digging about the avbuild/windows-desktop/ffmpeg-7.0-windows-desktop
and find out it doesn't supports the libx264
encoder:
so I test with multiple encoder and decided to use h264_mf
(Microsoft Media Foundation) which is supported by avbuild version of ffmpeg & its working:
Let me know if any problem occurs :)
@tribhuwan-kumar nice amazing! trusting you, merging PS: does audio transcription work for you on windows? |
Let me fix it in real quick! Update: I'm sorry, I created a func. in named |
Yes, the audio transcription is working, If any issue occurs i'll let you know. |
i tested with multiple video encoding for windows but none them are working like H.264 / MPEG 4 Part 10 encoding
eg:
VP8 encoding:
AV1 encoding
/claim #319
/fix: #319