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

Improve video decode performance on Electron #1567

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rafaellehmkuhl
Copy link
Member

Fix #1562

@rafaellehmkuhl
Copy link
Member Author

From Tony's first tests, this patch "improved performance by 99%" on Orange Pi 5.

He will be making some more tests with this version on Windows and macOS, to see if there are any downsides, or we can make them default. In any case we will allow the user to set them (and warn that they need to restart the application for them to take effect).

electron/main.ts Outdated
Comment on lines 24 to 25
app.commandLine.appendSwitch('flag-switches-begin')
app.commandLine.appendSwitch('flag-switches-end')
Copy link
Member

Choose a reason for hiding this comment

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

From some documentation:

These two flags are added around the switches chrome:flags adds to the command line. This is useful to see which switches were added by chrome:flags on chrome:version. They don't have any effect.

Since there is nothing around this flags, it does not make sense to have it here.

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes yes. This is a first test literally replicating the same arguments that the chromium version that Tony was running had, but those will be filtered.

@rafaellehmkuhl rafaellehmkuhl force-pushed the improve-video-decode-performance-electron branch from 31f868f to b0e5936 Compare January 14, 2025 20:22
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.

Enable hardware acceleration on Electron
2 participants