>>it should be downward compatible

>Sure of that?

No. I just thought about users, that already use discretization option to get 
mass of clumps (like me ...)
If the users now use the same script with new yade version, it will give 
different results (maybe unnoticed by the user).

>As far as inertia matters, there is no point in subdiscretizing this tiny 
>sphere which is 1000 times smaller that the clump.
>But if you really want to do it you can: discretization=10000.

Yes I see the problem and I agree with the change in code, but please
give a note for the users!

Now to the doc ... After a closer look I think there should be no change
in [1] since it just says that aabb is discretized (but not how). There
should be an update of [2] instead.

Also it needs an update of  the examples/clumps section ...

>p.s. Please don't cc me or anybody else when sending email to
bug/aswers/lists because it generates duplicates

I did not cc anyone. This seems to be a bug in launchpad mail delivery
system. I get double-posts nearly every day.

[1] see above
[2] 
https://yade-dem.org/doc/yade.wrapper.html?highlight=clump#yade.wrapper.BodyContainer.clump

-- 
You received this bug notification because you are a member of Yade
developers, which is subscribed to Yade.
https://bugs.launchpad.net/bugs/1474956

Title:
  Clump::updateProperties discretization not defined robustly

Status in Yade:
  Confirmed

Bug description:
  Most likely the reason for performance drop reported in [1]

  dx in the voxel decomposition is minRad/discretization.
  The problem is when a sphere of radius 1 is member of a clump which aabb has 
the size 1000 (for instance).
  Using the minimal discretization=1 gives a number of voxel = 1e9 in such case 
and it is no possible to go below this number.
  I think discretization should refer to aabb size, not member size. I have 
this fixed locally, let me know if you agree with the fix.

  [1] https://answers.launchpad.net/yade/+question/268954

To manage notifications about this bug go to:
https://bugs.launchpad.net/yade/+bug/1474956/+subscriptions

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

Reply via email to