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




_______________________________________________
Mailing list: https://launchpad.net/~yade-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~yade-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to