-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Character movement round 1 #881
base: main
Are you sure you want to change the base?
Conversation
…consider small distances as overlap.
…s in SIMD bias for capsule vs capsule to reduce feature flip-flop time of impact fallback now based on minExtent
sensors skip shapes on same body enableSensorEvents is back and defaults to false
Hello! I am testing this branch, this is really an improvement! Thanks! I would suggest to rename the new Anyway, thanks! |
bool isSensor; | ||
|
||
/// Enable contact events for this shape. Only applies to kinematic and dynamic bodies. Ignored for sensors. | ||
/// Enable sensor events for this shape. This applies to sensors and non-sensors. False by default, even for sensors. | ||
bool enableSensorEvents; |
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 would suggest to rename the new enableSensorEvents
flag to detectedBySensors
because at first I thought I had to enable this in the sensor itself, but rather it must be enabled in all the other shapes that could be detected by a sensor.
Hmm, after further testing, I see that Shouldn't it work so that For now, I am setting Thanks! |
Sensor updates:
[fix] body move events are now correctly adjusted for time of impact