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

Updated to latest wgpu (0.18.0) #3505

Merged
merged 5 commits into from
Nov 11, 2023
Merged

Updated to latest wgpu (0.18.0) #3505

merged 5 commits into from
Nov 11, 2023

Conversation

Wumpf
Copy link
Collaborator

@Wumpf Wumpf commented Oct 26, 2023

Tested on M1 Mac:

  • native
  • webgl, firefox
  • webgpu, chrome

all looking normal

Updated minor ahash version because 0.8.1 got yanked. Added some deny exceptions for now - we'll have to update winit soon to resolve glow related cargo deny errors (not a big issue though since we don't expect wgpu and glow backends to be used at the same time)

@Wumpf Wumpf added egui-wgpu dependencies Pull requests that update a dependency file labels Nov 1, 2023
@Wumpf Wumpf mentioned this pull request Nov 1, 2023
28 tasks
Copy link
Owner

@emilk emilk left a comment

Choose a reason for hiding this comment

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

Sad to see so many added/duplicated dependencies, but hopefully we can fix that once we update winit and glow

@emilk emilk merged commit cd46691 into master Nov 11, 2023
36 of 37 checks passed
@emilk emilk deleted the wgpu-0.18-update branch November 11, 2023 20:58
emilk added a commit that referenced this pull request Nov 16, 2023
@torokati44
Copy link
Contributor

May I ask, which release is this going to be the part of, and when can that be expected? 👀

@torokati44
Copy link
Contributor

The reason I asked, is that it would have been nice (for Ruffle at least) to get an egui version that updated wgpu and naga, but not yet winit and rwh.
It's not super important though, we can wait for 0.24 just fine.

@Wumpf
Copy link
Collaborator Author

Wumpf commented Nov 20, 2023

I heard @emilk saying that the plan is to release an egui version with this (but no new winit) later this week :)

emilk added a commit to rerun-io/rerun that referenced this pull request Dec 1, 2023
### What

* Fixes #3973
* Depends on emilk/egui#3505

### Test Matrix
Make sure to click through a few examples to cover at least points,
lines and meshes. (Arkit is a good one!)

* [ ] Test window-related stuff on Linux:
  * [ ] check that resizing the window works
  * [ ] that the icon shows
  * [ ] that file->open works
  * [ ] that the DPI is sane and persisted on restart
  * [ ] Look for any other weird regressions


* [x] Mac
    * [x] native
    * [x] WebGL Firefox
    * [x] WebGL Chrome
    * [x] WebGL Safari
    * [x] (optional) WebGPU Chrome
* [x] Linux
    * [x] native
    * [x] WebGL Firefox
    * [x] WebGL Chrome
* [x] Windows
    * [x] native
    * [x] WebGL Firefox
    * [x] WebGL Chrome
    * [x] WebGL Edge
    * [ ] (optional) WebGPU Chrome
* [x] Merge egui PR

### Checklist
* [x] emilk/egui#3505 has been merged
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)
* [x] I've included a screenshot or gif (if applicable)
* [x] I have tested [demo.rerun.io](https://demo.rerun.io/pr/4111) (if
applicable)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG

- [PR Build Summary](https://build.rerun.io/pr/4111)
- [Docs
preview](https://rerun.io/preview/73199e5727c18529cdbd2dd3956af17204403f96/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/73199e5727c18529cdbd2dd3956af17204403f96/examples)
<!--EXAMPLES-PREVIEW-->
- [Recent benchmark results](https://ref.rerun.io/dev/bench/)
- [Wasm size tracking](https://ref.rerun.io/dev/sizes/)

---------

Co-authored-by: Emil Ernerfeldt <emil.ernerfeldt@gmail.com>
Co-authored-by: Antoine Beyeler <antoine@rerun.io>
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.

3 participants