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.kn...@gmail.com>: > > 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.tho...@gmail.com>: > >> 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@lists.launchpad.net > >> > Unsubscribe : https://launchpad.net/~yade-dev > >> > More help : https://help.launchpad.net/ListHelp > >> > >> _______________________________________________ > >> Mailing list: https://launchpad.net/~yade-dev > >> Post to : yade-dev@lists.launchpad.net > >> Unsubscribe : https://launchpad.net/~yade-dev > >> More help : https://help.launchpad.net/ListHelp _______________________________________________ Mailing list: https://launchpad.net/~yade-dev Post to : yade-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~yade-dev More help : https://help.launchpad.net/ListHelp
