-
Notifications
You must be signed in to change notification settings - Fork 984
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
Implement ray query candidate intersection generation and confirmation #7047
base: trunk
Are you sure you want to change the base?
Conversation
9d50d75
to
8cbea73
Compare
970b0bf
to
16c902b
Compare
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.
I don't know about metal and dot's requirements but apart from this (and comments) it looks good! I'm not a real reviewer so their comments override mine.
@@ -2,23 +2,23 @@ | |||
|
|||
🧪Experimental🧪 | |||
|
|||
`wgpu` supports an experimental version of ray tracing which is subject to change. The extensions allow for acceleration structures to be created and built (with |
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.
It might not be helpful to have these formatting changes.
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.
Looks harmless though? Also, unintentional - my editor made them automatically.
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.
Yeah, but it made it slower to figure out what had changed. Personally I don't think this really needs to be fixed as it would be difficult but it might be worth turning that feature off in your editor if possible.
Connections
Fixes #6759
Description
Implements missing functionality for working with candidate intersections.
These functions are only functional in SPIR-V and HLSL.
Metal doesn't appear to expose this in the "old" API - we need to hook up their newer ray query objects.
Testing
Just shader validation. The function signatures are fairly straightforward.
Checklist
cargo fmt
.taplo format
.cargo clippy
. If applicable, add:--target wasm32-unknown-unknown
cargo xtask test
to run tests.CHANGELOG.md
. See simple instructions inside file.