-
Notifications
You must be signed in to change notification settings - Fork 34
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
Box to Box Collisions #937
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #937 +/- ##
==========================================
- Coverage 36.00% 35.83% -0.17%
==========================================
Files 293 296 +3
Lines 24079 24192 +113
==========================================
Hits 8669 8669
- Misses 15410 15523 +113 ☔ View full report in Codecov by Sentry. |
96c7f77
to
cb165f5
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clang-Tidy
found issue(s) with the introduced code (1/1)
92e8909
to
6a471f4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clang-Tidy
found issue(s) with the introduced code (1/1)
120c02c
to
676c498
Compare
No Clang-Tidy warnings found so I assume my comments were addressed
a1f3d40
to
6a458ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM! Just some nitpicks to fix before merging :)
engine/include/cubos/engine/collisions/narrow_phase/colliding_with.hpp
Outdated
Show resolved
Hide resolved
11848a5
to
ef1f98f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clang-Tidy
found issue(s) with the introduced code (1/1)
ef1f98f
to
7edb861
Compare
No Clang-Tidy warnings found so I assume my comments were addressed
Description
Implement narrow phase collision detection for box vs box.
Checklist