Re: [Yade-dev] [Yade-users] Optimized collider needs testing
...e no, compilation is breaking now, independent if you activate cgal or not. First, yade was giving an error on runtime if I didn't activate cgal feature. Klaus On Wed, 28 Sep 2011 06:55:50 pm Bruno Chareyre wrote: > > Does it break compilation without cgal? Oh dear... Then it's another bug > > Fixed. > > > ___ > 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 -- Dr. Klaus Thoeni - Centre for Geotechnical and Materials Modelling Civil, Surveying and Environmental Engineering - Engineering Building EA The University of Newcastle, Callaghan, NSW 2308, Australia web: http://www.newcastle.edu.au/research-centre/cgmm phone: +61 (0)2 4921 5118 ___ 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
Re: [Yade-dev] [Yade-users] Optimized collider needs testing
Yes, it is listed. You have just to to change Branches with status: Any status. That's how I found out that there are 2 collide branches. Klaus On Wed, 28 Sep 2011 06:01:29 pm Bruno Chareyre wrote: > Ok thanks. It explains. > > lp:~bruno-chareyre/yade/collide2 contains further improvements (and code > cleaning) as compared to lp:~bruno-chareyre/yade/collide, which was > already faster than trunk. But before yesterday, collide2 was simply trunk. > > collide without the "2" should not exist actually, it's not listed here > for instance: https://code.launchpad.net/yade > I'm surprised that you could download it. > > Bruno > > On 28/09/11 04:49, Klaus Thoeni wrote: > > Hi Bruno, > > > > I checked out your branch lp:~bruno-chareyre/yade/collide (without 2 at > > the end as indicated in the bzr co part of your first posting) when I > > tested it. I just saw now that there is as well a branch > > lp:~bruno-chareyre/yade/collide2. Probably that's the one to test. This > > explains why it was working. I will do the tests with this branch now. > > > > So guys if you want to test Bruno's new implementation make sure you > > check out the right branch: > > bzr checkout lp:~bruno-chareyre/yade/collide2 > > > > Cheers, > > Klaus > > > > On Tue, 27 Sep 2011 10:43:28 pm Bruno Chareyre wrote: > >> There was indeed a bug with clumps (fixed), but the thing I don't > >> understand is the branch did not contain the changes. Bazaar branches > >> are not quite user-friendly for commits/updates... > >> Klaus, you probably didn't try the good code (but then how did you spot > >> the bug? I'm lost...). > >> Could you send me a copy of the core/bound.hpp you downloaded, just to > >> be sure? > >> > >> Bruno > >> > >> On 26/09/11 10:37, Bruno Chareyre wrote: > >>> (moving to yade-dev) > >>> > >>> No guess yet. Thanks for notifying. Which scripts gives the segfault? > >>> > >>> B > >>> > >>> On 26/09/11 07:43, Klaus Thoeni wrote: > Hi Bruno, > > I tested your implementation with my examples (dynamic problem with > about 1 particles) and the computational time in > InsertionSortCollider slightly improved as expected. > > However, when running your branch with examples with clumps gives a > Segmentation Fault. Any guess? > > Klaus > > On Tue, 20 Sep 2011 07:29:02 AM Bruno Chareyre wrote: > > Hi all, > > > > I worked on some optimizations of contact detection recently and I > > would like to get feedback from you to detect possible bugs and > > confirm the speedup for different applications. If everything is ok, > > the changes can be part of the next release after some code cleanup > > and documentation. > > > > If you have a chance, could you please download and compile the > > branch lp:~bruno-chareyre/yade/collide2 (bzr checkout > > lp:~bruno-chareyre/yade/collide, cd yade, scons, as usual) and try > > this with your own problems? > > I would be gratefull if you could report differences in terms of > > computation speed. > > I don't expect big improvements for small quasistatic problems. The > > speedup is more likely to happen on large and/or dynamic problems, > > where it should range between x2 and x3, or even more in multithread. > > Some results are reported here: > > https://yade-dem.org/wiki/Colliders_performace#Improved_InsertionSort > > > > Thanks. > > > > Bruno -- Dr. Klaus Thoeni - Centre for Geotechnical and Materials Modelling Civil, Surveying and Environmental Engineering - Engineering Building EA The University of Newcastle, Callaghan, NSW 2308, Australia web: http://www.newcastle.edu.au/research-centre/cgmm phone: +61 (0)2 4921 5118 ___ 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
Re: [Yade-dev] [Yade-users] Optimized collider needs testing
> Does it break compilation without cgal? Oh dear... Then it's another bug Fixed. ___ 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
[Yade-dev] [Bug 850864] Re: strange facet-sphere behaviour
I agree Anton. The question is how to fix that? It needs knowledge of facets connectivity I think, the same kind of problems I got to tackle for chained cylinder. It is not quite well supported by design. In the cylinder-sphere case, interactions have additional data (e.g. an additional id3) and the connectivity of cylinders is stored in static members of the class ChainedCylinder. (https://blueprints.launchpad.net/yade/+spec/chained-bodies) Another way would be to define a collection of facet as a unique body to make sure there is only one contact (the problem is there can be multiple contacts on concave shapes...). It would need to assign multiple bboxes to a single body, too. Can you imagine simpler fix? Any thought on this question can be usefull since we (in Grenoble) want to implement deformable membranes on this basis one day. -- You received this bug notification because you are a member of Yade developers, which is the registrant for Yade. https://bugs.launchpad.net/bugs/850864 Title: strange facet-sphere behaviour Status in Yet Another Dynamic Engine: Confirmed Bug description: See https://answers.launchpad.net/yade/+question/171223 Spheres that are exactly on an edge between facets behave strangely. Maybe related to other (closed, but never solved) bugs in sphere- facet. To manage notifications about this bug go to: https://bugs.launchpad.net/yade/+bug/850864/+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
Re: [Yade-dev] [Yade-users] Optimized collider needs testing
Does it break compilation without cgal? Oh dear... Then it's another bug sorry (ZECollider is the problem I guess?), cgal is not mandatory. I'll document the new algorithm to explain parameters, so you can tune the collider for different problems. How much speedup do you have in your simulations? B On 28/09/11 08:17, Klaus Thoeni wrote: > Hi guys, > > don't forget to enable feature cgal before compiling ;-) > > I checked my test with the implementation in branch lp:~bruno- > chareyre/yade/collide2. Computational speed improves compared to trunk > version. Very good! And clumps are working as well. > > Good job Bruno! > > Cheers > Klaus > > On Tue, 27 Sep 2011 10:43:28 pm Bruno Chareyre wrote: >> There was indeed a bug with clumps (fixed), but the thing I don't >> understand is the branch did not contain the changes. Bazaar branches >> are not quite user-friendly for commits/updates... >> Klaus, you probably didn't try the good code (but then how did you spot >> the bug? I'm lost...). >> Could you send me a copy of the core/bound.hpp you downloaded, just to >> be sure? >> >> Bruno >> >> On 26/09/11 10:37, Bruno Chareyre wrote: >>> (moving to yade-dev) >>> >>> No guess yet. Thanks for notifying. Which scripts gives the segfault? >>> >>> B >>> >>> On 26/09/11 07:43, Klaus Thoeni wrote: Hi Bruno, I tested your implementation with my examples (dynamic problem with about 1 particles) and the computational time in InsertionSortCollider slightly improved as expected. However, when running your branch with examples with clumps gives a Segmentation Fault. Any guess? Klaus On Tue, 20 Sep 2011 07:29:02 AM Bruno Chareyre wrote: > Hi all, > > I worked on some optimizations of contact detection recently and I > would like to get feedback from you to detect possible bugs and > confirm the speedup for different applications. If everything is ok, > the changes can be part of the next release after some code cleanup > and documentation. > > If you have a chance, could you please download and compile the branch > lp:~bruno-chareyre/yade/collide2 (bzr checkout > lp:~bruno-chareyre/yade/collide, cd yade, scons, as usual) and try this > with your own problems? > I would be gratefull if you could report differences in terms of > computation speed. > I don't expect big improvements for small quasistatic problems. The > speedup is more likely to happen on large and/or dynamic problems, > where it should range between x2 and x3, or even more in multithread. > Some results are reported here: > https://yade-dem.org/wiki/Colliders_performace#Improved_InsertionSort > > Thanks. > > Bruno -- ___ 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 ___ 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
Re: [Yade-dev] [Yade-users] Optimized collider needs testing
Ok thanks. It explains. lp:~bruno-chareyre/yade/collide2 contains further improvements (and code cleaning) as compared to lp:~bruno-chareyre/yade/collide, which was already faster than trunk. But before yesterday, collide2 was simply trunk. collide without the "2" should not exist actually, it's not listed here for instance: https://code.launchpad.net/yade I'm surprised that you could download it. Bruno On 28/09/11 04:49, Klaus Thoeni wrote: > Hi Bruno, > > I checked out your branch lp:~bruno-chareyre/yade/collide (without 2 at the > end as indicated in the bzr co part of your first posting) when I tested it. > I > just saw now that there is as well a branch lp:~bruno-chareyre/yade/collide2. > Probably that's the one to test. This explains why it was working. I will do > the tests with this branch now. > > So guys if you want to test Bruno's new implementation make sure you check > out > the right branch: > bzr checkout lp:~bruno-chareyre/yade/collide2 > > Cheers, > Klaus > > On Tue, 27 Sep 2011 10:43:28 pm Bruno Chareyre wrote: >> There was indeed a bug with clumps (fixed), but the thing I don't >> understand is the branch did not contain the changes. Bazaar branches >> are not quite user-friendly for commits/updates... >> Klaus, you probably didn't try the good code (but then how did you spot >> the bug? I'm lost...). >> Could you send me a copy of the core/bound.hpp you downloaded, just to >> be sure? >> >> Bruno >> >> On 26/09/11 10:37, Bruno Chareyre wrote: >>> (moving to yade-dev) >>> >>> No guess yet. Thanks for notifying. Which scripts gives the segfault? >>> >>> B >>> >>> On 26/09/11 07:43, Klaus Thoeni wrote: Hi Bruno, I tested your implementation with my examples (dynamic problem with about 1 particles) and the computational time in InsertionSortCollider slightly improved as expected. However, when running your branch with examples with clumps gives a Segmentation Fault. Any guess? Klaus On Tue, 20 Sep 2011 07:29:02 AM Bruno Chareyre wrote: > Hi all, > > I worked on some optimizations of contact detection recently and I > would like to get feedback from you to detect possible bugs and > confirm the speedup for different applications. If everything is ok, > the changes can be part of the next release after some code cleanup > and documentation. > > If you have a chance, could you please download and compile the branch > lp:~bruno-chareyre/yade/collide2 (bzr checkout > lp:~bruno-chareyre/yade/collide, cd yade, scons, as usual) and try this > with your own problems? > I would be gratefull if you could report differences in terms of > computation speed. > I don't expect big improvements for small quasistatic problems. The > speedup is more likely to happen on large and/or dynamic problems, > where it should range between x2 and x3, or even more in multithread. > Some results are reported here: > https://yade-dem.org/wiki/Colliders_performace#Improved_InsertionSort > > Thanks. > > Bruno -- ___ 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 ___ 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
Re: [Yade-dev] [Yade-users] Optimized collider needs testing
Hi guys, don't forget to enable feature cgal before compiling ;-) I checked my test with the implementation in branch lp:~bruno- chareyre/yade/collide2. Computational speed improves compared to trunk version. Very good! And clumps are working as well. Good job Bruno! Cheers Klaus On Tue, 27 Sep 2011 10:43:28 pm Bruno Chareyre wrote: > There was indeed a bug with clumps (fixed), but the thing I don't > understand is the branch did not contain the changes. Bazaar branches > are not quite user-friendly for commits/updates... > Klaus, you probably didn't try the good code (but then how did you spot > the bug? I'm lost...). > Could you send me a copy of the core/bound.hpp you downloaded, just to > be sure? > > Bruno > > On 26/09/11 10:37, Bruno Chareyre wrote: > > (moving to yade-dev) > > > > No guess yet. Thanks for notifying. Which scripts gives the segfault? > > > > B > > > > On 26/09/11 07:43, Klaus Thoeni wrote: > >> Hi Bruno, > >> > >> I tested your implementation with my examples (dynamic problem with > >> about 1 particles) and the computational time in > >> InsertionSortCollider slightly improved as expected. > >> > >> However, when running your branch with examples with clumps gives a > >> Segmentation Fault. Any guess? > >> > >> Klaus > >> > >> On Tue, 20 Sep 2011 07:29:02 AM Bruno Chareyre wrote: > >>> Hi all, > >>> > >>> I worked on some optimizations of contact detection recently and I > >>> would like to get feedback from you to detect possible bugs and > >>> confirm the speedup for different applications. If everything is ok, > >>> the changes can be part of the next release after some code cleanup > >>> and documentation. > >>> > >>> If you have a chance, could you please download and compile the branch > >>> lp:~bruno-chareyre/yade/collide2 (bzr checkout > >>> lp:~bruno-chareyre/yade/collide, cd yade, scons, as usual) and try this > >>> with your own problems? > >>> I would be gratefull if you could report differences in terms of > >>> computation speed. > >>> I don't expect big improvements for small quasistatic problems. The > >>> speedup is more likely to happen on large and/or dynamic problems, > >>> where it should range between x2 and x3, or even more in multithread. > >>> Some results are reported here: > >>> https://yade-dem.org/wiki/Colliders_performace#Improved_InsertionSort > >>> > >>> Thanks. > >>> > >>> Bruno -- Dr. Klaus Thoeni - Centre for Geotechnical and Materials Modelling Civil, Surveying and Environmental Engineering - Engineering Building EA The University of Newcastle, Callaghan, NSW 2308, Australia web: http://www.newcastle.edu.au/research-centre/cgmm phone: +61 (0)2 4921 5118 ___ 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