yade-dev team mailing list archive
-
yade-dev team
-
Mailing list archive
-
Message #07963
Re: [Bug 729079] Re: Performance optimization of InsertionSortCollider (Anton)
Yes, this is expected: the new is better with more threads and also with
more bodies. How many bodies in your simulation?
The fact that you find worst speed for j=1 is annoying though... Until
now I only found new speed at least equal to old, but you found some
cases where new < old.
It makes the choice of moving to the new algorithm difficult, or it
suggests to have two colliding logics but it is uneasy since it also
affects Newton, aabb dispatching, interaction container...
We need however to consider the tuning of parameters. Maybe the default
values I defined are not good in your case and another optimum can be
found. If you could send me a simplified script I could test. Or for a
simple test, you could just decrease verletDist. The default is 0.9, you
could try the same as in the old algorithm: 0.05, and see what happens.
Bruno
On 18/10/11 18:23, Anton Gladky wrote:
> Ok, seems, warnings are gone.
>
> Speed, iter/sec
> -j Old collider New_Collider:
> 1 7.94 6.7
> 2 13.85 12.89
> 3 14.83 16.11
>
> The new collider works better with threads more than 2.
>
> Anton
>
>
>
>
> On Mon, Oct 17, 2011 at 3:57 PM, Bruno Chareyre
> <bruno.chareyre@xxxxxxxxxxx> wrote:
>> Hi Anton,
>>
>> I commited fixes for the collider (branch collide2, r2925) and for cgal
>> feature. I don't find any problem when removing bodies now, but let you
>> try with your scripts.
>> Now, all interactions should be removed correctly after deleting bodies.
>>
>> Bruno
>>
> _______________________________________________
> Mailing list: https://launchpad.net/~yade-dev
> Post to : yade-dev@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~yade-dev
> More help : https://help.launchpad.net/ListHelp
>
--
_______________
Bruno Chareyre
Associate Professor
ENSE³ - Grenoble INP
11, rue des Mathématiques
BP 46
38402 St Martin d'Hères, France
Tél : +33 4 56 52 86 21
Fax : +33 4 76 82 70 43
________________
Follow ups
References