Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add main menu setting for browser GPU rendering #3816

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

Merged
Dutchman101 merged 5 commits into multitheftauto:master from Lpsd:cef-gpu-setting
Oct 20, 2024

Conversation

@Lpsd
Copy link
Member

@Lpsd Lpsd commented Oct 19, 2024
edited
Loading

Adds two options to "web browser" tab in the main menu settings:

  • Enable GPU rendering (CVAR: browser_enable_gpu=1)
  • Enable GPU compositing (CVAR: browser_enable_gpu_compositing=1)

image

GPU rendering must be enabled for GPU compositing to be available. Both options require a restart upon change to take effect.

Adds a clientside Lua function to check if browser GPU is enabled:

bool isBrowserGPUEnabled()

Note: for some reason the GPU is not being disabled for me, even though I've debugged and can see the disable-gpu flag being applied (http://gta.rockstarvision.com/vehicleviewer still works for example, which relies on WebGL). Need others to test and see if they get the same results, as it doesn't make much sense 🤔

GPU has always been disabled in MTA, until #2933 (merged 2 weeks ago)

Update: according to @theSarrum these changes work (both enabling and disabling), tested by attempting to load a page with WebGL content.

Proxy-99 and Fernando-A-Rocha reacted with thumbs up emoji Fernando-A-Rocha reacted with heart emoji
@Dutchman101 Dutchman101 merged commit bfdfdb5 into multitheftauto:master Oct 20, 2024
6 checks passed
MTABot pushed a commit that referenced this pull request Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@TracerDS TracerDS TracerDS approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /