G1=E1/(2*(1+nu1))

[Hongyang Cheng]

Hi,

I am a Yade user. The discussion [1] brings me here.

I browse to the source code file for Hertz contact model [2] which you posted on [1].

In line #27, the equation to compute shear modulus should be G=E/(2*(1+nu)) if I am not mistaken. It seems you drop a parenthesis there.

My apology if I take this wrong. BTW thanks for those detailed and logical comments with the source code.

Cheers, Alex

[1] https://answers.launchpad.net/yade/+question/235934)
[2] http://bazaar.launchpad.net/~eudoxos/woo/trunk/view/head:/pkg/dem/Hertz.cpp

[Václav Šmilauer (eudoxos)]

Hi, sorry, thanks for looking at that. You're right, of course the parenthesis is missing (2*1 would be better written as 2, right?...). I did not see your post right away, somehow launchpad did not notify me. It is fixed in rev. 3605 which I just landed to launchpad. Thanks!

[Hongyang Cheng (alex-cheng)]

You are very welcome. It’s my pressure to contribute a little to your big
project. :-)
Looking forward for new developments in woo.

On Wed, Dec 3, 2014 at 8:05 PM, Václav Šmilauer <email address hidden> wrote:

> ** Changed in: woo
> Status: New => Fix Released
>
> ** Changed in: woo
> Importance: Undecided => High
>
> ** Changed in: woo
> Assignee: (unassigned) => Václav Šmilauer (eudoxos)
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1385563
>
> Title:
> G1=E1/(2*(1+nu1))
>
> Status in Woo - discrete dynamic computations:
> Fix Released
>
> Bug description:
> Hi,
>
> I am a Yade user. The discussion [1] brings me here.
>
> I browse to the source code file for Hertz contact model [2] which you
> posted on [1].
>
> In line #27, the equation to compute shear modulus should be
> G=E/(2*(1+nu)) if I am not mistaken. It seems you drop a parenthesis
> there.
>
> My apology if I take this wrong. BTW thanks for those detailed and
> logical comments with the source code.
>
> Cheers, Alex
>
> [1] https://answers.launchpad.net/yade/+question/235934)
> [2]
> http://bazaar.launchpad.net/~eudoxos/woo/trunk/view/head:/pkg/dem/Hertz.cpp
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/woo/+bug/1385563/+subscriptions
>