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

Height Field Support #39

Open
tay10r opened this issue Oct 5, 2021 · 2 comments
Open

Height Field Support #39

tay10r opened this issue Oct 5, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@tay10r
Copy link

tay10r commented Oct 5, 2021

Any interest in supporting acceleration structures for height fields?

There's a paper that goes into describing the process. I'm currently using this library for calculating ambient occlusion in terrains, and this approach may lead to a performance boost.

Even so much as a dedicated BVH builder for height fields would be really cool to see.

@madmann91
Copy link
Owner

Hello! I've started a new job and I am really busy at the moment, but if you feel confident doing this, feel free to send a pull request. I'm not sure if that would apply here, but note that any shape that has a bounding box should already be supported, essentially by implementing a custom primitive that fulfills the contract that the BVH builders and ray intersection kernel expects.

@madmann91 madmann91 added the enhancement New feature or request label Oct 13, 2021
@tay10r
Copy link
Author

tay10r commented Oct 13, 2021

Congrats on the new job! I'll implement it and see if the build/traversal times are worth it, compared to the existing methods.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants