yade-dev team mailing list archive

[Chiara Modenese <c.modenese@xxxxxxxxx>]

Yes, there has been a shift (exactly as you say). I suggested to add one
more option (homo=0 is, in fact, doing nothing) to follow Cundall/Thornton's
way. To summarize (as in doc -
http://beta.arcig.cz/~eudoxos/yade/sphinx/yade.wrapper.html?highlight=homodeform#yade.wrapper.Cell.homoDeform,
but I need to repeat it here to stress my question).


homo=0 -> nothing
homo=1 -> Cundall's way (we update particle positions, incorrectly atm, see
the bug)
homo=2 -> update particle velocities (old homo=1)
homo=3 -> update particle velocities, including the convective term (old
homo=2)


I have two questions, one of which is for Bruno. My first point is that I do
not understand the origin of the convective term (see the discussion we had
in the thread I mentioned before) that you actually introduced. The second
point is that I do not know if the Cundall's way (homo=1) is the same thing
as what you essentially coded (homo=2,3). I wanted to make a comparison,
that is why I opened the bug.


Chiara


On 12 November 2010 17:43, Chareyre <674000@xxxxxxxxxxxxxxxxxx> wrote:

> Chiara, it seems to me that the flag homoDeform have been shifted. The
> old homo=1 and homo=2 is now homo=2 and homo=3.
> I don't know what the new homo=1 means, so I can't really explain what
> happens in that case.
>
> --
> homoDeform=1 does not handle the periodic case correctly.
> https://bugs.launchpad.net/bugs/674000
> You received this bug notification because you are a direct subscriber
> of the bug.
>
> Status in Yet Another Dynamic Engine: New
>
> Bug description:
> Currently, in the periodic case, if homoDeform=1 (alias Cundall way) we do
> not adjust the values of the relative velocities as according to the
> velocity gradient. This should be done, since, in such a case, we only
> update particle positions and not particle velocities.
>
> To unsubscribe from this bug, go to:
> https://bugs.launchpad.net/yade/+bug/674000/+subscribe
>

-- 
homoDeform=1 does not handle the periodic case correctly.
https://bugs.launchpad.net/bugs/674000
You received this bug notification because you are a member of Yade
developers, which is subscribed to Yade.

Status in Yet Another Dynamic Engine: New

Bug description:
Currently, in the periodic case, if homoDeform=1 (alias Cundall way) we do not adjust the values of the relative velocities as according to the velocity gradient. This should be done, since, in such a case, we only update particle positions and not particle velocities.