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

web: update web-sys to 0.3.58, sparse attachments support #2813

Merged
merged 1 commit into from Jun 28, 2022

Conversation

jinleili
Copy link
Contributor

@jinleili jinleili commented Jun 27, 2022

Connections
#2795
sparse attachments support need web-sys upgrade: #2778

Description
From web-sys 0.3.58, call GpuRenderPipelineDescriptor GpuComputePipelineDescriptor::new() need a "non-null" layout argument, so when desc.layout is none, wasm_bindgen::JsValue::from(web_sys::GpuAutoLayoutMode::Auto) is used and tested on Chrome Canary.

Testing
Tested cube hellow-triangle hellow-compute mipmap masa-line examples on Chrome Canary (v105.0.5144.0), Other examples have many warnings in the Console panel, but none of them are relevant to this update.

@jinleili
Copy link
Contributor Author

jinleili commented Jun 27, 2022

Oh, and I noticed a CI failure that doesn't look like it was caused by this PR?

--- STDERR:                    wgpu::example/hello-compute tests::test_multithreaded_compute ---
[467](https://github.com/gfx-rs/wgpu/runs/7065980115?check_suite_focus=true#step:13:468)
error: XDG_RUNTIME_DIR not set in the environment.
[468](https://github.com/gfx-rs/wgpu/runs/7065980115?check_suite_focus=true#step:13:469)
error: XDG_RUNTIME_DIR not set in the environment.
[469](https://github.com/gfx-rs/wgpu/runs/7065980115?check_suite_focus=true#step:13:470)
WARNING: lavapipe is not a conformant Vulkan implementation, testing use only.

@cwfitzgerald
Copy link
Member

That one's been a rare but spurious failure, so just ignore it

@jinleili jinleili changed the title Update backend/web.rs to match web-sys v0.3.58 web: update web-sys to 0.3.58, sparse attachments support Jun 28, 2022
@cwfitzgerald
Copy link
Member

Want to give it a once over on my computer tomorrow, but I don't see any major issues.

@jinleili
Copy link
Contributor Author

Another code change that looks awkward is clear_value,web-sys have been aligned to the WebGPU Spec but wgpu have not. Next PR, I will split clear_value from LoadOp::Clear(Color) to match spec.

Copy link
Member

@cwfitzgerald cwfitzgerald left a comment

Choose a reason for hiding this comment

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

LGTM

@cwfitzgerald cwfitzgerald merged commit 0eb6845 into gfx-rs:master Jun 28, 2022
@jinleili jinleili deleted the web-sys branch June 28, 2022 22:43
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.

None yet

2 participants