-
Notifications
You must be signed in to change notification settings - Fork 23
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 rotation to integration #1305
base: main
Are you sure you want to change the base?
Conversation
|
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)
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1305 +/- ##
==========================================
- Coverage 36.25% 36.20% -0.05%
==========================================
Files 395 396 +1
Lines 31537 31578 +41
==========================================
Hits 11433 11433
- Misses 20104 20145 +41 ☔ View full report in Codecov by Sentry. |
d054db1
to
56900aa
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.
LGTM!
@luishfonseca will only be able to review this PR in 5 days. |
I should probably be removed from the list of reviewers during P1, I'll have more time again on P2 onwards |
Description
Apply torque and angular impulses on integration to rotate the bodies.
Resources: https://research.ncl.ac.uk/game/mastersdegree/gametechnologies/physicstutorials/3angularmotion/Physics%20-%20Angular%20Motion.pdf
Checklist