Re: [Yade-dev] Yade with CGAL 4.11

2017-11-29 Thread Bruno Chareyre
On 11/29/2017 03:25 PM, Janek Kozicki wrote: I hope that you have, Bruno, among those tests some related to PFV and CGAL :) Hi Janek, Yes we have, thanks. :) DEM-PFV-check.py depends on pfv+cgal+TesselationWrapper+openblas+cholmod (at least...). You can use it as a test if it helps. I

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-28 Thread Janek Kozicki (yade-dev)
Uh, Bruno :-) I was thinking to ask you whether you have given my diff to Anton after you have modified it. Please do it, or just confirm that the one which I sent is good to go (which isn’t optimal due to too many #ifdefs as you mentioned earlier). best regards Janek On 27 Nov 2017, 20:04

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-27 Thread Anton Gladky
Hi all, is there any progress on fixing this issue? I have disabled CGAL for Debian builds, but it would be good to switch it on again. Also the stable Yade version is almost one year old. It is time to release something newer soon. If there are some volunteers to go through existing bugs on

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-07 Thread Bruno Chareyre
On 11/07/2017 06:47 PM, Janek Kozicki wrote: I just feel a little uneasy about cross_product of two (sphere-sphere) pairs ;) Well... after admitting that spheres can be mutually orthogonal (which is really the case) cross product is fine... :) But you are right, here cross product is

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-07 Thread Janek Kozicki
Very good! I was about to check that :) I should have done that in the first place, my mistake :) Also, indeed even these extra `.point()` could be eliminated if several of those functions were redeclared as overload with another signature. I just feel a little uneasy about cross_product of two

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-07 Thread Bruno Chareyre
I can confirm my previous message: all the changes should compile with older CGAL. The only place where an ifdef is needed maybe is when including "constructions_on_weighted_points_cartesian_3.h". Bruno On 11/07/2017 11:04 AM, Bruno Chareyre wrote: Awesome. Whenever implicit conversion from

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-07 Thread Bruno Chareyre
Awesome. Whenever implicit conversion from wp (weighted point) to bp (bare point) is used in current code it is just because there is no reason to type v->point().point() when we can write simply v->point(). However, what we need is really a bp, so the v->point().point() syntax reflects more

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-06 Thread Janek Kozicki
In hindsight there has to be a better way, instead of all those defines. If we cannot have an implicit conversion we could create a function which takes argument type of Point_3 or a type of Weighted_point_3 and always returns Point_3. Then the fixes scattered around the code would not depend on

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-06 Thread Bruno Chareyre
On 11/03/2017 09:13 PM, Anton Gladky wrote: I will probably remove libcgal-dev dependency from the package right now not to block CGAL migration. If we find a solution, it can easily be resumed later. Sounds good. Thank you Anton. B ___ Mailing

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-05 Thread Janek Kozicki
Thank you guys, the removal of --no-parallel was especially useful :-) It is still compiling as I write it, but it seems that I have managed to fix it. In short: they have changed the internal CGAL architecture to treat weighted points. The first error indicated that there is no implicit

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-03 Thread Anton Gladky
Hi Janek, parallel Yade build in Debian/Ubuntu is disabled due to enormous RAM consumption during build which leads to compilation failure. Remove "--no-parallel" option here [1] and you will be able to build in parallel mode. I tried to fix it myself, but without any positive result. I will

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-03 Thread Bruno Chareyre
Hi Janek, I don't think I'll be able to test this very quickly myself. Based on the compile logs the evident problem is [1]. Which sounds like a trivial issue. However, it may be the case that they really changed the design of the so-called geometry kernel (as suggested by [2]) and that it

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-03 Thread Janek Kozicki
Hi, I am building with `time dpkg-buildpackage -rfakeroot -b -j32` but it appears that -j32 is deliberately ignored. I understand why, but my PC is powerful enough ;) Can you tell me how to enable it back? It must be somewhere in ./debian/* dir that you have prepared. best regards Janek

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-03 Thread Janek Kozicki
Hi Anton, did you try to fix it yourself during past week? (I see that this bug report is a week old) If yes, do you have maybe some useful advice? Am I correct that this FTBFS is about yade in (clean install of) stretch with CGAL 4.11 pulled from experimental? best regards Janek Anton Gladky

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-02 Thread Luc Scholtes
Hum... Don't think so since I have cgal 4.7 installed on my computer. Luc 2017-11-02 13:34 GMT-07:00 Robert Caulk : > Could this be the source of the strange PFV behaviors, Luc? > > On Thu, Nov 2, 2017 at 1:22 PM, Anton Gladky > wrote: > >> Hi all,

Re: [Yade-dev] Yade with CGAL 4.11

2017-11-02 Thread Robert Caulk
Could this be the source of the strange PFV behaviors, Luc? On Thu, Nov 2, 2017 at 1:22 PM, Anton Gladky wrote: > Hi all, > > it looks like Yade is not compatible with the new CGAL 4.11 and > it blocks the upload of new CGAL into the Debian [1]. > > It would be good if

[Yade-dev] Yade with CGAL 4.11

2017-11-02 Thread Anton Gladky
Hi all, it looks like Yade is not compatible with the new CGAL 4.11 and it blocks the upload of new CGAL into the Debian [1]. It would be good if somebody could try to fix this issue. [1] https://bugs.debian.org/876524 Thanks Anton ___ Mailing