yade-dev team mailing list archive
-
yade-dev team
-
Mailing list archive
-
Message #10052
[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