-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
Upgrade to wgpu 28 #22265
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
base: main
Are you sure you want to change the base?
Upgrade to wgpu 28 #22265
Changes from all commits
710b654
8f9aa8c
ee743f8
d487417
75d0a0b
3ee13c6
5dda602
d60c0cd
fcb2e31
f76b7f3
f497164
fd8d4e5
067b4b9
c53d84c
fe99a1e
0753f20
18c5d68
a9c6f2a
bdacbf0
3b2b655
bb5f440
e222549
1789194
53db3a5
2712514
694270b
5d3ffe7
cf30512
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -77,12 +77,13 @@ impl ViewNode for DlssNode<DlssSuperResolutionFeature> { | |
| command_encoder.push_debug_group("dlss_super_resolution"); | ||
| let time_span = diagnostics.time_span(command_encoder, "dlss_super_resolution"); | ||
|
|
||
| dlss_context | ||
| let dlss_command_buffer = dlss_context | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why is this necessary? I might be missing something
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Mixing the wgpu encoding API with the raw encoding API is not longer allowed, so dlss_wgpu needed some changes, and this reflects the new usage. |
||
| .render(render_parameters, command_encoder, &adapter) | ||
| .expect("Failed to render DLSS Super Resolution"); | ||
|
|
||
| time_span.end(command_encoder); | ||
| command_encoder.pop_debug_group(); | ||
| render_context.add_command_buffer(dlss_command_buffer); | ||
|
|
||
| Ok(()) | ||
| } | ||
|
|
@@ -153,13 +154,15 @@ impl ViewNode for DlssNode<DlssRayReconstructionFeature> { | |
| command_encoder.push_debug_group("dlss_ray_reconstruction"); | ||
| let time_span = diagnostics.time_span(command_encoder, "dlss_ray_reconstruction"); | ||
|
|
||
| dlss_context | ||
| let dlss_command_buffer = dlss_context | ||
| .render(render_parameters, command_encoder, &adapter) | ||
| .expect("Failed to render DLSS Ray Reconstruction"); | ||
|
|
||
| time_span.end(command_encoder); | ||
| command_encoder.pop_debug_group(); | ||
|
Comment on lines
161
to
162
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think we need to move these to a new command encoder after add_command_buffer. We basically need 3 command encoders:
However I think we'll need some changes to bevy's diagnostics API in order to make this work, as currently it expects you to use the same encoder to start/end a timestamp query iirc. I can work on this after this PR, but just wanted to note it. |
||
|
|
||
| render_context.add_command_buffer(dlss_command_buffer); | ||
|
|
||
| Ok(()) | ||
| } | ||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to land upstream and have a release before we can merge this PR.