Re: [Yade-dev] Naming GridConnection

2016-06-17 Thread Klaus Thoeni
Yes, will do! On Thu, 16 Jun 2016 07:30:52 PM Anton Gladky wrote: > Hi Klaus, > > could you please update also a test-case. Because during yade --test > one get a notification " chainedCylinder is deprecated." > > Thanks > > Anton > > > 2016-06-10 14:43 GMT+02:00 Klaus Thoeni

[Yade-dev] Naming GridConnection

2016-06-16 Thread Anton Gladky
Hi Klaus, could you please update also a test-case. Because during yade --test one get a notification " chainedCylinder is deprecated." Thanks Anton 2016-06-10 14:43 GMT+02:00 Klaus Thoeni >: > Hi guys, > > just pulled some helper functions for

Re: [Yade-dev] Naming GridConnection

2016-06-16 Thread François
2016-06-16 10:28 GMT+02:00 Bruno Chareyre : > > > On 06/16/2016 09:22 AM, François wrote: > > > - GridNodes have no bounding-box, we can't expect them to interact > > Or, maybe, we could exploit the collision detection between connexions and > boxes to instantiate

Re: [Yade-dev] Naming GridConnection

2016-06-16 Thread François
Hi Klaus, And thanks for this work ! My answer below : 2016-06-10 14:43 GMT+02:00 Klaus Thoeni : > Hi guys, > > just pulled some helper functions for cylinders and some related examples. > > I also created a script examples/cylinders/cylinderconnection-roots.py > which >

Re: [Yade-dev] Naming GridConnection

2016-06-10 Thread Klaus Thoeni
Hi guys, just pulled some helper functions for cylinders and some related examples. I also created a script examples/cylinders/cylinderconnection-roots.py which is aimed to replace examples/chained-cylinders/chained-cylinder-roots.py. I couldn't get the box-gridConnection interactions to

Re: [Yade-dev] Naming GridConnection

2016-05-17 Thread François
2016-05-11 15:11 GMT+02:00 Bruno Chareyre : > > > On 05/11/2016 01:52 PM, Klaus Thoeni wrote: > >> >> I think sooner or later we should get rid of the current ChainedCylinder >> implementation. >> > Agreed > >> But you are right, the algorithms are slightly

Re: [Yade-dev] Naming GridConnection

2016-05-11 Thread Bruno Chareyre
On 05/11/2016 01:52 PM, Klaus Thoeni wrote: I think sooner or later we should get rid of the current ChainedCylinder implementation. Agreed But you are right, the algorithms are slightly different, although it might just the physics (?). I haven't compared the code yet but I added two

Re: [Yade-dev] Naming GridConnection

2016-05-11 Thread Klaus Thoeni
Hi again, > ChainedCylinder could also print a warning, doing nothing in the simulation > but suggesting to have a look at GridNodes/Connections documentation. > Cylinders and Grid don't have exactly the same algorithms, therefore I > think it is safer to do nothing rather than letting the user

Re: [Yade-dev] Naming GridConnection

2016-05-11 Thread François
An other "problem" is linked to the grid-grid contacts : Grids elements need a set of specific masks and the collider "avoidSelfInteractionMask" to be set, OR to add the GridCo-GridCo IGeom/Law2 in the engines. 2016-05-11 13:33 GMT+02:00 François : > Hi, >

Re: [Yade-dev] Naming GridConnection

2016-05-11 Thread François
Hi, ChainedCylinder could also print a warning, doing nothing in the simulation but suggesting to have a look at GridNodes/Connections documentation. Cylinders and Grid don't have exactly the same algorithms, therefore I think it is safer to do nothing rather than letting the user think it is

Re: [Yade-dev] Naming GridConnection

2016-05-11 Thread Klaus Thoeni
Hi Bruno, here my suggestion: add Cylinder and ChainedCylinder functions to gridpfacet.py. Both functions would not require to define GridNodes to avoid confusion but they would be generated by the functions (hidden from the user). Grids would still require the generation of GridNodes. What

[Yade-dev] Naming GridConnection

2016-05-11 Thread Bruno Chareyre
Hi all, Reading Klaus' answer below I realize that replacing "cylinder" by "grid connection" will not help to intuitively recognize which class can be used to modelize a single cylinder. I wonder how it could be improved. Beyhond renaming GridConnection (still an option), we could make