← Back to team overview

yade-dev team mailing list archive

[Bug 1229783] Re: Loading clumps takes too much time

 

> Why then the loop FOREACH (line 162) does not stop after the first
> sphere has been found?

because you found a bug!
^^

thanks anton, it is hopefully fixed here:

https://github.com/yade/trunk/commit/4f66770c9fdd2bf9e2af07a7bfe690a7d4ad9be7

> BTW, the loop over the grid could probably be parallelised with some
> #pragma omp parallel for reduction(+:M,Sg,Ig).

If your C++ skills are high enough, please do it!

> Of course, that would be nice, something like appendClumped(listOfBodies,
> divisor). I was actually about to have a look how to implement it as well. So
> let me know if you are doing it.

It is not that easy, because updateProperties is called by 5 methods,
not just appendClumped (see above)!

divisor must be defined globally somewhere in the code (i do not know
where is a good spot). Also we can change the default value (actually
15) if you think it is too high.

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

Title:
  Loading clumps takes too much time

Status in Yet Another Dynamic Engine:
  In Progress

Bug description:
  The examples/packs/packs.py hangs in line 46 [1] because
  of appendClumped command.

  Changing it to "append" fixes the problem. So, it seems, 
  appendClump has infinite loop  somewhere.

  [1]
  https://github.com/yade/trunk/blob/master/examples/packs/packs.py#L46

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


References