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

modify "set_nearby" method to allow AABB to point to any number of nearby AABBs instead of just one #37

Open
digitalajf opened this issue Jun 19, 2020 · 0 comments
Labels
enhancement New feature or request invalid This doesn't seem right

Comments

@digitalajf
Copy link
Owner

digitalajf commented Jun 19, 2020

since ver 3.16

Currently, the "set_nearby" method will simply have the calling AABB object point to a given AABB provided to the method as a paramater:

public void set_Nearby(AABB nrby)
{
   nearby = nrby;
} 

This method was meant to simply test the Quadtree's ability to determine nearby AABBs by having the calling AABB mark the first one it encounters through the search as being "nearby". In reality, this is illogical in that there can be many AABBs that may be determined to be nearby at any given time throughout the simulation. Nearby AABBs are already marked by a Quadnode at the deepest tree depth. That is how the Quadtree works after all. Instead of using a "nearby" marker which marks the first AABB that is tested to be in close proximity, each AABB that is marked can reference the common Quadnode that intersects all of them.

@digitalajf digitalajf added bug Something isn't working enhancement New feature or request labels Jun 19, 2020
@digitalajf digitalajf changed the title modify "set_nearby" method to allow AABB to point to any number of nearby AABBs instead of just one. modify "set_nearby" method to allow AABB to point to any number of nearby AABBs instead of just one Jun 19, 2020
@digitalajf digitalajf added invalid This doesn't seem right and removed bug Something isn't working labels Oct 19, 2021
@digitalajf digitalajf added this to the Major Enhancement 1 milestone Oct 19, 2021
@digitalajf digitalajf pinned this issue Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request invalid This doesn't seem right
Development

No branches or pull requests

1 participant