Agreed, no need for this condition.
"revertSign" could also be removed now.
In fact the design seems loose to me. You ask for fabric tensor, you
trigger a bunch of weak/strong related things and even some silent
settings of GUI variables... Calculating the fabric tensor takes less
1/3 of the total work it seems.
Bruno
On 03/28/2016 09:32 PM, Anton Gladky wrote:
Hi Jerome,
I do not see also anything periodic-specific there, so from my
POV that condition can be removed. We can also ask Chiara
about that.
Best regards
Anton
2016-03-28 21:13 GMT+02:00 Jerome Duriez <[email protected]
<mailto:[email protected]>>:
Hello,
What's the reason for fabricTensor() utils function to be designed
only for periodic simulations (according to the doc and the error
raised in aperiodic conditions:
https://github.com/yade/trunk/blob/c5330ac6a612db52275b10f87691b5630ab8c24a/pkg/dem/Shop_02.cpp#L262)
?
I could not find any and was thinking to remove this condition
(keeping a test to disregard contact with boundaries ?), but
preferred to ask first.
Jerome
_______________________________________________
Mailing list: https://launchpad.net/~yade-dev
<https://launchpad.net/%7Eyade-dev>
Post to : [email protected]
<mailto:[email protected]>
Unsubscribe : https://launchpad.net/~yade-dev
<https://launchpad.net/%7Eyade-dev>
More help : https://help.launchpad.net/ListHelp
_______________________________________________
Mailing list: https://launchpad.net/~yade-dev
Post to : [email protected]
Unsubscribe : https://launchpad.net/~yade-dev
More help : https://help.launchpad.net/ListHelp
--
_______________
Bruno Chareyre
Associate Professor
ENSE³ - Grenoble INP
Lab. 3SR
BP 53
38041 Grenoble cedex 9
Tél : +33 4 56 52 86 21
Fax : +33 4 76 82 70 43
________________
_______________________________________________
Mailing list: https://launchpad.net/~yade-dev
Post to : [email protected]
Unsubscribe : https://launchpad.net/~yade-dev
More help : https://help.launchpad.net/ListHelp
