Skip to content
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

Advanced FFMpeg configuration: Hardware acceleration, codec and quality choice. #1485

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

AntonIXO
Copy link


name: Advanced FFMpeg configuration: Hardware acceleration, codec and quality choice.

description

Added the ability to change codec, enable hardware acceleration and change quality.

how to test

  1. check screenpipe --help
  2. test options: --video-codec {codec} --hw-accel {hw} --video-crf {num} --video-preset {preset}
  3. Check the settings page in UI

Copy link

vercel bot commented Feb 25, 2025

@AntonIXO is attempting to deploy a commit to the louis030195's projects Team on Vercel.

A member of the Team first needs to authorize it.

@@ -81,6 +86,11 @@ export function DataImportSection() {
creation_time: new Date().toISOString(),
fps: 30,
device_name: "",
video_codec: "",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does the add cli uses these settings?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, so I removed these lines

@@ -900,7 +920,129 @@ export function RecordingSettings() {
</div>
</div>

{/* */}
<div className="flex flex-col space-y-4">
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i wonder if we can make the UI more scalable, like, most people don't want to bother with those, so maybe a advanced setting part or idk

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved down to an advanced settings menu

@AntonIXO
Copy link
Author

I had to merge Wayland fix also to test changes

@AntonIXO
Copy link
Author

AntonIXO commented Mar 5, 2025

@louis030195 Do you have plans to merge it? Made changes to settings and moved it to an advanced menu. Wayland fix seems not needed anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants