Skip to content

aabb and sphere intersect functions#56

Merged
recp merged 8 commits into
masterfrom
sphere
Jun 9, 2018
Merged

aabb and sphere intersect functions#56
recp merged 8 commits into
masterfrom
sphere

Conversation

@recp
Copy link
Copy Markdown
Owner

@recp recp commented May 29, 2018

AABB functions:

  • AABB vs AABB
  • AABB vs Point
  • AABB vs Sphere
  • AABB contains AABB

New features for sphere:

  • Sphere Transform, Merge (same as AABB helpers)
  • Sphere vs Sphere
  • Sphere vs Point

I used glm_aabb_aabb for testing intersection, it could be glm_aabb_intersect or glm_aabb_intr but since for instance glm_aabb_point is used instead of glm_aabb_intersect_point same convention is also used for aabb.

Sphere Representation in cglm is vec4: [center.x, center.y, center.z, radii], functions will accept vec4 instead of two parameter for each sphere e.g . vec3 for center and float for radius. It is combined into vec4 in cglm.

If you have better idea or don't like this design please let me know.

recp added 2 commits May 29, 2018 23:19
* AABB vs AABB
* AABB vs Point
* AABB vs Sphere
* AABB contains AABB
@coveralls
Copy link
Copy Markdown

coveralls commented May 29, 2018

Coverage Status

Coverage decreased (-0.5%) to 13.166% when pulling 857265b on sphere into af812e8 on master.

@recp recp merged commit 93e6c3c into master Jun 9, 2018
@recp recp deleted the sphere branch June 9, 2018 15:44
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