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

Implement ray query candidate intersection generation and confirmation #7047

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

kvark
Copy link
Member

@kvark kvark commented Feb 1, 2025

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

  • Run cargo fmt.
  • Run taplo format.
  • Run cargo clippy. If applicable, add:
    • --target wasm32-unknown-unknown
  • Run cargo xtask test to run tests.
  • Add change to CHANGELOG.md. See simple instructions inside file.

@kvark kvark requested a review from a team as a code owner February 1, 2025 09:00
@kvark kvark force-pushed the rt-confirm branch 3 times, most recently from 9d50d75 to 8cbea73 Compare February 1, 2025 09:06
@kvark kvark force-pushed the rt-confirm branch 4 times, most recently from 970b0bf to 16c902b Compare February 1, 2025 17:53
Copy link
Contributor

@Vecvec Vecvec left a 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
Copy link
Contributor

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.

Copy link
Member Author

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.

Copy link
Contributor

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.

etc/specs/ray_tracing.md Show resolved Hide resolved
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.

Allow for comitting candidate intersections
3 participants