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

Does not work when app has a separate process (i.e. GPU Helper) #23

Open
marceliwac opened this issue Dec 8, 2020 · 1 comment
Open

Comments

@marceliwac
Copy link

There are apps that use additional processes for GPU acceleration (i.e. Microsoft Teams, Slack, Chrome) where this script doesn't really help. It is however pretty easy to fix this, as these processes are usually contained within <something GPU Helper>.app files, so it's just a matter of selecting these too.

As an example, Google Chrome uses an additional GPU Helper process:

image

Setting the checkbox Prefer external GPU for just the Google Chrome.app does not do the trick, because the actual GPU acceleration is ran by the other process. What has worked for me was manually changing that process' checkbox for external GPU preference. Having tested it with other apps, all of these seem to follow the pattern bellow:

image
image
image

Essentially, it's a matter of getting into the application's contents folder and scanning for *.app files.

I would be happy to work on this when I have time, but that might not be for a while. Given how simple this is I would be happy for someone else to take over, should they have the time.

@andrewgodman
Copy link

I can take a look at this as it's a bit annoying.

Do you think it's common sense we should only add the GPU helper process for now in chromium and electron apps or adding any other .apps would be the best approach?

I think the script will have to run again on each app update?

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

No branches or pull requests

2 participants