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 option verbose debugging to dynamic BVH. #44650

Closed
wants to merge 1 commit into from

Conversation

lawnjelly
Copy link
Member

In order to test tree balancing, this prints out the tree after certain operations. This is enabled at compile time only, by setting the DYNAMICBVH_VERBOSE_DEBUG define at the top of the header.

Notes

  • This can either be merged as is or used as an example for adding debugging, or we can keep debugging out of the class. However with trees of these sort it is usually useful to have some debugging support.
  • This debugging shows the current BVH is not working correctly, see Replace Octree by DynamicBVH in cull code #44623

In order to test tree balancing, this prints out the tree after certain operations. This is enabled at compile time only.
@lawnjelly
Copy link
Member Author

Forgot to close this, I believe there are plans to port the bvh from godot 3.x to 4.0 so adding features to this version may no longer be desirable (we can always resurrect if plans change etc).

@lawnjelly lawnjelly closed this May 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants