Skip to content

Conversation

Wumpf
Copy link
Collaborator

@Wumpf Wumpf commented Oct 2, 2025

Tested on:

  • MacOS native
  • Windows native
  • Firefox @ GL
  • Chrome @ WebGPU

@Wumpf Wumpf requested a review from lucasmerlin as a code owner October 2, 2025 15:30
@Wumpf Wumpf added egui-wgpu dependencies Pull requests that update a dependency file labels Oct 2, 2025
Copy link

github-actions bot commented Oct 2, 2025

Preview available at https://egui-pr-preview.github.io/pr/7580-update-wgpu
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

use crate::texture_to_image::texture_to_image;

/// Timeout for waiting on the GPU to finish rendering.
pub(crate) const WAIT_TIMEOUT: Duration = Duration::from_secs(1);
Copy link
Owner

Choose a reason for hiding this comment

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

This sounds maybe too low, like in case of a shared instance yadda yadda

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hmm is it though? On windows your driver hard-resets after 2 seconds (unless you change settings in the Windows Registry)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The anomaly we seem to be recently hitting is that on Mac an undetected error occurs, the GPU actually no longer does work and then we get stuck indefinitely.

Not sure what you mean with shared instance :)

Base automatically changed from update-msrv to main October 2, 2025 17:12
@Wumpf Wumpf merged commit 427c076 into main Oct 3, 2025
47 checks passed
@Wumpf Wumpf deleted the update-wgpu branch October 3, 2025 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file egui-wgpu

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants