Yade

the option homoDeform=1 is incorrect

Bug #836867 reported by Chiara Modenese on 2011-08-29

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Yade	Fix Released	Undecided	Unassigned

Bug Description

Let's remember us that homoDeform=1 of the periodic case is not correctly implemented. The option would be meaningful if relative velocity computed in ScGeom was also updated according to the velocity related to the strain rate of the periodic space (Cundall's way). As it is now, it is NOT to be used.

Revision history for this message

Bruno Chareyre (bruno-chareyre) wrote on 2011-09-15:

This is a result of my non-intrusive development style... I implement correct equations as an option. Maybe I should also erase the wrong ones? ;)
I suggest to simply remove the homodeform parameter and make the current homo=3 the only possible behaviour.
What about that?

Revision history for this message

Chiara Modenese (chiara-modenese) wrote on 2011-09-15:

Yes, I agree with that. It would have been nice to have a comparison with Cundall's way but as it is now it is not consistent and am not sure if anyone has time to fix it.

Revision history for this message

Bruno Chareyre (bruno-chareyre) wrote on 2011-09-15: Re: [Yade-dev] [Bug 836867] Re: the option homoDeform=1 is incorrect

On the top of that, no one can tell how Cundall actually implemented the
equations...
Ok, I'll do that.

Revision history for this message

Bruno Chareyre (bruno-chareyre) wrote on 2011-09-20:

fixed in r2922