search

EulalieCoevoet / AdaptiveMerging

With adaptive merging we reduce computation time in rigid body simulations.
https://eulaliecoevoet.github.io/AdaptiveMerging/
42 stars 11 forks source link

lambdas inside contact #17

Closed paulkry closed 5 years ago

paulkry commented 5 years ago

Instead of storing lambdas in a vector, allocating these vectors, and keeping contact indices, can we just let these values live inside the contacts? Not a bug, but certainly something that will help prevent bugs!

EulalieCoevoet commented 5 years ago

I wanna do this. Tell me if someone is already on it.

otmanon commented 5 years ago

All yours!

paulkry commented 5 years ago

Yes, do it! Should make a number of things cleaner in the code, with the only drawback being that it is harder to see the full lambda vector... but we never were doing such things.

EulalieCoevoet commented 5 years ago

Done in commit. Looks like it didn't break nor fix anything :D