[Bepu2] IPoseInegratorCallbacks-IntegrateVelocity-bodyIndex

Post and discuss features you'd like to see in the BEPUphysics library.
Post Reply
Darkon76
Posts: 23
Joined: Mon Mar 27, 2017 12:33 pm

[Bepu2] IPoseInegratorCallbacks-IntegrateVelocity-bodyIndex

Post by Darkon76 »

Hello,

Adding damping per object I notice that IPoseIntegratorCallbacks -> IntegrateVelocity -> bodyIndex isn't the index of the affected body it is the index inside the ActiveSet.

The variable name is confusing.
Norbo
Site Admin
Posts: 4929
Joined: Tue Jul 04, 2006 4:45 am

Re: [Bepu2] IPoseInegratorCallbacks-IntegrateVelocity-bodyIndex

Post by Norbo »

Across v2, the term "index" refers to the current location in memory within whatever buffer is relevant. So in this case, index is the correct term. The pose integrator iterates over all bodies in the contiguous active set, so the index is the most directly useful value.

"Handle" refers to the identity of an object. At the moment, both are int types, which I don't like because it leads to confusion like this. I considered using a dedicated type for handles to avoid the issue, but at the time there was no zero overhead way to do it. I'll probably revisit this in the future since this class of confusion (and potentially bug) is going to be so common. Even if there is a slight overhead, a type-level distinction for handles is probably worth it.
Post Reply