← Back to team overview

yade-dev team mailing list archive

Re: Naming GridConnection

 

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
>

Follow ups

References