Should we `clear` inside `collide`?

[lmontaut]

Should we `clear` inside `collide`?

[vicltz]

I had the same issue, debug was very hard because of this behavior.
Maybe it's a good idea to add the clear inside the distance or collide function ?

[jcarpent]

Regarding this issue, we can add an extra argument to collide and distance, named bool clear_results = true, in such a way people can still do both.

[lmontaut]

I guess we can but I just don't see the reason why we don't simply clear inside distance or collide. If the results are not cleared, it means that the results inside QueryResult have no relation to one another, because they may have been called on different poses, or worse, different collision pairs... Maybe there is a use case I am missing but I just don't see the point of not clearing. Multiple people have reported this issue to me already and it's a super annoying bug to track down when someone is not familiar with hpp-fcl.

[lmontaut]

EDIT: I think the rationale behind not clearing is that if you set the num_max_contacts in CollisionRequest, you can collect up to num_max_contacts on different collision pairs with the same CollisionResult.
On the contrary, I am very used to creating a single CollisionResult for each collision pair. If I have non-convex objects like BVHs then num_max_contacts allows me to collect multiple contact points; if I have convex objects, I know I will get only one contact, so I set num_max_contacts to 1.

Anyway, I think mixing collision pairs inside the same result is a bad idea, and the current design of hpp-fcl enforces that.

@jcarpent I will do a PR with your solution and set the default behavior to clear the result