Skip to content

Update wgpu to a commit that fixes use on Nvidia drivers#98

Merged
mmstick merged 1 commit into
masterfrom
wgpu-update
Feb 6, 2024
Merged

Update wgpu to a commit that fixes use on Nvidia drivers#98
mmstick merged 1 commit into
masterfrom
wgpu-update

Conversation

@ids1024
Copy link
Copy Markdown
Member

@ids1024 ids1024 commented Feb 6, 2024

This can be tested with something like VK_ICD_FILENAMES=/usr/share/vulkan/icd.d/nvidia_icd.json cargo run -p tour --features iced/wgpu.

On Nvidia I'm seeing a flood of Suboptimal present of frame warnings. So some improvement may still be needed here. But if it doesn't regress behavior on other hardware, that seems like an improvement over freezing.

This can be tested with something like
`VK_ICD_FILENAMES=/usr/share/vulkan/icd.d/nvidia_icd.json cargo run -p tour
--features iced/wgpu`.

On Nvidia I'm seeing a flood of `Suboptimal present of frame` warnings.
So some improvement may still be needed here. But if it doesn't regress
behavior on other hardware, that seems like an improvement over
freezing.
@ids1024 ids1024 requested a review from a team February 6, 2024 17:21
@wash2
Copy link
Copy Markdown

wash2 commented Feb 6, 2024

Is it 20fda69 specifically that has an important fix? I have just been adding a patch for wgpu to all of the projects that I plan on removing when there is a release with your fix, but I used faed98b45cdc105df78b110e3fd6ca3eef36efc7

@ids1024
Copy link
Copy Markdown
Member Author

ids1024 commented Feb 6, 2024

20fda69 is just the latest commit to wgpu. Which seems as good an option as any.

Generally iced-based apps shouldn't depend on wgpu except through iced, so it seems best to change the dep here, in one place (well 2; also glyphon), rather than adding [patch] .

@mmstick mmstick merged commit 575de00 into master Feb 6, 2024
@mmstick mmstick deleted the wgpu-update branch February 6, 2024 17:46
ids1024 added a commit to pop-os/libcosmic that referenced this pull request Feb 6, 2024
mmstick pushed a commit to pop-os/libcosmic that referenced this pull request Feb 6, 2024
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.

3 participants