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

Make WgpuSettings::default() check WGPU_POWER_PREF #9482

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

fornwall
Copy link
Contributor

Objective

Allow users to specify the power preference when selecting a wgpu adapter, which is useful for testing or workaround purposes, and makes the behaviour consistent with the already present check for WGPU_BACKEND.

Solution

In WgpuSettings::default(), allow users to specify the WGPU_POWER_PREF to affect the wgpu adapter choice.

@github-actions
Copy link
Contributor

Welcome, new contributor!

Please make sure you've read our contributing guide and we look forward to reviewing your pull request shortly ✨

@fornwall fornwall changed the title Make WgpuSettings::default() obevy WGPU_POWER_PREF Make WgpuSettings::default() check WGPU_POWER_PREF Aug 18, 2023
This is useful when testing, and makes the behaviour consistent
with the already present check for WGPU_BACKEND.
@github-actions
Copy link
Contributor

Example alien_cake_addict failed to run, please try running it locally and check the result.

1 similar comment
@github-actions
Copy link
Contributor

Example alien_cake_addict failed to run, please try running it locally and check the result.

@mockersf mockersf added A-Rendering Drawing game state to the screen S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it labels Aug 18, 2023
@mockersf mockersf added this pull request to the merge queue Aug 18, 2023
Merged via the queue into bevyengine:main with commit 80db794 Aug 18, 2023
21 of 22 checks passed
@fornwall fornwall deleted the wgpu-power-pref-env branch August 18, 2023 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Rendering Drawing game state to the screen S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants