yade-dev team mailing list archive

[Bruno Chareyre <bruno.chareyre@xxxxxxxxxxx>]

> I don't get why you changed this, just for the norm()?. It breaks my
> code.

It's ok. Unless I made a mistake it doesn't change anything in you code since I didn't modify the original function. The new function has the only puprpose to be faster, it is consistent to remove this "norm()".

> 	geom->scaleToDisplacementTMax(multiplier);
>
> is manifestly incorrect, as multiplier is a dimensionless force ratio,
> whereas displacementT is, well, displacement, a length.
>
>   
True. The point was just to get feedback, not to find the best name. I 
should have commit the comment saying that scaleToDisplacement DOES NOT 
return sleeped distance, since the caller is supposed to know it already 
(he needs that to pass the multiplier).

Bruno