Skip to content
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

Cannot read property 'x' of null in FaceClipper.reduce #32

Closed
cjbailey opened this issue May 2, 2020 · 2 comments
Closed

Cannot read property 'x' of null in FaceClipper.reduce #32

cjbailey opened this issue May 2, 2020 · 2 comments
Labels

Comments

@cjbailey
Copy link

cjbailey commented May 2, 2020

It seems possible that min1, min2, min1V and min2V sometimes aren't being set (and remain null), leading to the error mentioned in the title.

This seems to happen during BoxBoxDetector. Spheres and Boxes appear to collide just fine.

TypeError Cannot read property 'x' of null oimo_collision_narrowphase_detector__$BoxBoxDetector_FaceClipper.reduce https://j4k75.csb.app/node_modules/oimophysics/OimoPhysics.js:1:156619 oimo_collision_narrowphase_detector_BoxBoxDetector.detectImpl https://j4k75.csb.app/node_modules/oimophysics/OimoPhysics.js:1:145380 oimo_collision_narrowphase_detector_BoxBoxDetector.detect https://j4k75.csb.app/node_modules/oimophysics/OimoPhysics.js:1:122639 oimo_dynamics_Contact._updateManifold https://j4k75.csb.app/node_modules/oimophysics/OimoPhysics.js:1:374223 oimo_dynamics_World._updateContacts https://j4k75.csb.app/node_modules/oimophysics/OimoPhysics.js:1:416406 oimo_dynamics_World.step https://j4k75.csb.app/node_modules/oimophysics/OimoPhysics.js:1:615737

@saharan saharan added the bug label May 20, 2021
@saharan
Copy link
Owner

saharan commented May 20, 2021

Thank you for reporting.
It looks like this may happen when vertices went infinity for some reason...

@saharan
Copy link
Owner

saharan commented Jan 24, 2022

fixed in 1.2.2.

@saharan saharan closed this as completed Jan 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants