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

Ray Tracing Tracking Issue #6762

Open
5 of 24 tasks
Vecvec opened this issue Dec 16, 2024 · 4 comments
Open
5 of 24 tasks

Ray Tracing Tracking Issue #6762

Vecvec opened this issue Dec 16, 2024 · 4 comments
Labels
api: dx12 Issues with DX12 or DXGI api: metal Issues with Metal api: vulkan Issues with Vulkan area: api Issues related to API surface feature: raytracing Issues with the Ray Tracing Native Feature type: enhancement New feature or request

Comments

@Vecvec
Copy link
Contributor

Vecvec commented Dec 16, 2024

For the current implementation see https://github.com/gfx-rs/wgpu/blob/trunk/etc/specs/ray_tracing.md.
For prior details see #1040

Acceleration structures

Features

Nice to have (less important features / underlying APIs still in development)

  • Micromaps (opacity and displacement)

Design

  • Reconsider build_acceleration_structures_unsafe_tlas after as_hal has been added.

Known Bugs

Support

Ray Queries

Features

Design

Known Bugs

Support

Ray Tracing Pipeline

@cwfitzgerald cwfitzgerald added type: enhancement New feature or request area: api Issues related to API surface api: dx12 Issues with DX12 or DXGI api: metal Issues with Metal api: vulkan Issues with Vulkan feature: raytracing Issues with the Ray Tracing Native Feature labels Dec 16, 2024
@baadc0de
Copy link

baadc0de commented Jan 5, 2025

@Vecvec I'd like to get involved on this feature. #6609 seems like branch needs fixed to be compatible with upstream and tests fixed, is it a good starting point?

@Vecvec
Copy link
Contributor Author

Vecvec commented Jan 6, 2025

#6609 is in an odd situation - Mesa (which CI uses) has a bug in it that causes the test added to seg-fault, this bug is caused by resetting certain types of query pools, this has meant that that PR is not able to be merged (as it would cause everyone else's CI to fail). I was hoping that the patch would be merged soon, but that doesn't seem to be happening. Most of the other CI issues are caused by merging with trunk, and seem to be trivial.

@Vecvec
Copy link
Contributor Author

Vecvec commented Jan 6, 2025

Thanks for wanting to help! If you want to figure out how acceleration structure building works then implementing updating sometimes instead of building would be helpful too, it's probably also quite advanced though so a simpler thing to do would be general checking of code - the entire building process is quite large so it's likely that there are bugs in there somewhere. There are also #6803 and #6759 which would be nice to have implemented. General helping out is also useful, and so is testing the code using either the testing API or by writing code to use this.

@baadc0de
Copy link

baadc0de commented Jan 6, 2025

Thanks for wanting to help! If you want to figure out how acceleration structure building works then implementing updating sometimes instead of building would be helpful too, it's probably also quite advanced though so a simpler thing to do would be general checking of code - the entire building process is quite large so it's likely that there are bugs in there somewhere. There are also #6803 and #6759 which would be nice to have implemented. General helping out is also useful, and so is testing the code using either the testing API or by writing code to use this.

Thanks, I'll look at the linked issues and check the acc structure building code for robustness.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: dx12 Issues with DX12 or DXGI api: metal Issues with Metal api: vulkan Issues with Vulkan area: api Issues related to API surface feature: raytracing Issues with the Ray Tracing Native Feature type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants