yade-dev team mailing list archive

[Klaus Thoeni <klaus.thoeni@xxxxxxxxx>]

 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 think it is
> working as before.

I think sooner or later we should get rid of the current ChainedCylinder 
implementation. 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 examples in examples/grids which are the same as in 
examples/chained-cylinders and the results are slightly different. Any idea how 
many people are using ChainedCylinder?

> 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.

Yes, but this is something which should be pointed out by appropriate 
examples.

Cheers
K


> 2016-05-11 13:33 GMT+02:00 François <francois.kneib@xxxxxxxxx>:
> > 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
> > working as before.
> > 
> > François
> > 
> > 2016-05-11 13:23 GMT+02:00 Klaus Thoeni <klaus.thoeni@xxxxxxxxx>:
> >> 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 do you think? Let me know, I am happy to implement it.
> >> 
> >> Cheers
> >> Klaus
> >> 
> >> On Wed, 11 May 2016 12:04:44 PM Bruno Chareyre wrote:
> >> > 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 ChainedCylinder, or maybe just
> >> > Cylinder
> >> > an alias of GridConnection.
> >> > Other suggestions?
> >> > 
> >> > Bruno
> >> > 
> >> > On 05/06/2016 03:32 AM, Klaus Thoeni wrote:
> >> > > Question #293342 on Yade changed:
> >> > > https://answers.launchpad.net/yade/+question/293342
> >> > > 
> >> > > Klaus Thoeni proposed the following answer:
> >> > > Hi Alex,
> >> > > 
> >> > > in addition ChainedCylinder is deprecated and might be removed in the
> >> > > future. You should use GridConnection instead to create cylinders.
> >> > > For
> >> > > examples see [1]
> >> > > 
> >> > > K
> >> > > 
> >> > > [1] https://github.com/yade/trunk/tree/master/examples/grids
> >> > 
> >> > _______________________________________________
> >> > Mailing list: https://launchpad.net/~yade-dev
> >> > Post to     : yade-dev@xxxxxxxxxxxxxxxxxxx
> >> > Unsubscribe : https://launchpad.net/~yade-dev
> >> > More help   : https://help.launchpad.net/ListHelp
> >> 
> >> _______________________________________________
> >> Mailing list: https://launchpad.net/~yade-dev
> >> Post to     : yade-dev@xxxxxxxxxxxxxxxxxxx
> >> Unsubscribe : https://launchpad.net/~yade-dev
> >> More help   : https://help.launchpad.net/ListHelp