-
Notifications
You must be signed in to change notification settings - Fork 10
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
polycollection2d fix #130
polycollection2d fix #130
Conversation
steven11sjf
commented
Mar 14, 2024
- im stupid, turns out it's a vec3 for the AABOX's vPos's offset and it also applies to POLYCOLLECTION2D and presumably CIRCLE as well...
- im stupid, turns out it's a vec3 for the vPos's offset and it also applies to POLYCOLLECTION2D and presumably CIRCLE as well...
called it |
@@ -38,14 +38,11 @@ | |||
|
|||
collision_formats = { | |||
"AABOX2D": Struct( | |||
center=CVector2D, | |||
unk=Float, | |||
offset=CVector3D, |
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.
call these position
imo
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.
i was going for "offset from vPos", since it felt like center/position was a bit misleading. i can maybe just throw in a small comment to clarify instead.
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.
it's relative position, as opposed to global. in my experience relative position is the default when you just say "position"
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.
fair enough, fixed
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #130 +/- ##
=======================================
Coverage 74.13% 74.13%
=======================================
Files 63 63
Lines 3155 3155
=======================================
Hits 2339 2339
Misses 816 816 ☔ View full report in Codecov by Sentry. |