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

Add support for acceleration structures and ray queries #2213

Merged
merged 9 commits into from
Jun 2, 2023

Conversation

Rua
Copy link
Contributor

@Rua Rua commented May 29, 2023

Changelog:

### Breaking changes
- Index buffers are now specified using the `IndexBuffer` enum instead of the `Index` trait.

### Additions
- Added support for the `khr_acceleration_structure` and `khr_ray_query` extensions.

This finally adds support for acceleration structures and ray queries! It's a pretty big addition, with a lot of potential for errors, so I hope that people will try it out and help test it before 0.34 is released. There's a lot of unsafe here, because many things rely on data in device buffers that Vulkano can't check.

The acceleration structure extension also includes support for executing AS operations on the host instead of the device. But I decided not to implement those, because very few drivers actually support it at the moment, so it would be impossible to test.

@marc0246
Copy link
Contributor

marc0246 commented Jun 2, 2023

I see you named all the new Info-constructors new unlike in the rest of the library. I'm assuming this isn't just a copy/paste error and it's because some fields have very long names and couple that with some Info structs not having defaults for multiple fields.

@marc0246
Copy link
Contributor

marc0246 commented Jun 2, 2023

All LGTM, thanks so much! 🎉

@marc0246 marc0246 merged commit 7ccba17 into vulkano-rs:master Jun 2, 2023
marc0246 added a commit that referenced this pull request Jun 2, 2023
Rua added a commit that referenced this pull request Jun 2, 2023
@Rua Rua deleted the acceleration-structure branch October 25, 2023 14:25
hakolao pushed a commit to hakolao/vulkano that referenced this pull request Feb 20, 2024
)

* Add support for acceleration structures and ray queries

* Add support for acceleration structures and ray queries

* Add add_context helper method

* Remove Option from ValidationError::requires_one_of

* Update vulkano/src/command_buffer/commands/bind_push.rs

Co-authored-by: marc0246 <40955683+marc0246@users.noreply.github.com>

* Use id counter, small clippy fixes elsewhere

* Use byte buffers for things with strides

---------

Co-authored-by: marc0246 <40955683+marc0246@users.noreply.github.com>
hakolao pushed a commit to hakolao/vulkano that referenced this pull request Feb 20, 2024
hakolao pushed a commit to hakolao/vulkano that referenced this pull request Feb 20, 2024
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.

2 participants