yade-dev team mailing list archive

[Jerome Duriez <Jerome.Duriez@xxxxxxxxxxx>]

See https://github.com/yade/trunk/commit/c1a5e87a84423686780299b2fc09acfb2606a67b, thanks !

Jerome


________________________________
From: Yade-dev [yade-dev-bounces+jerome.duriez=ucalgary.ca@xxxxxxxxxxxxxxxxxxx] on behalf of Bruno Chareyre [bruno.chareyre@xxxxxxxxxxxxxxx]
Sent: May-26-16 7:05 AM
To: yade-dev@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Yade-dev] Dynamic bodies and getSpheresVolume()

It makes sense indeed.
The dynamic check was probably a loose evaluation of shape, now redundant.
Bruno

On 05/20/2016 11:30 PM, Jerome Duriez wrote:
Hi,

getSpheresVolume() currently skips all dynamic bodies before even considering whether they are spheres or not [*]. I think it would you make sense to just consider the shape, and to remove this "dynamic" condition.

It would for instance allow to use consistently porosity() function for a fixed packing of sphere. (Now, porosity() then always returns 1 in such case)

What do you think ?


Jerome


[*] https://github.com/yade/trunk/blob/6d4a0d1689c6bad2dbdd6e27d1653418f7fc6498/pkg/dem/Shop_01.cpp#L308


PS: the same applies for other functions, such as getSpheresMass(), but this has not been an issue for me...



_______________________________________________
Mailing list: https://launchpad.net/~yade-dev
Post to     : yade-dev@xxxxxxxxxxxxxxxxxxx<mailto:yade-dev@xxxxxxxxxxxxxxxxxxx>
Unsubscribe : https://launchpad.net/~yade-dev
More help   : https://help.launchpad.net/ListHelp