Vincent Rouvreau
Vincent Rouvreau
Tested on master and on a branch without #1102 and you are right, it fixes the issue. I am closing it, thanks !
* With `cython < 3.0.0`: ``` void function() except + nogil ^ ------------------------------------------------------------ file.pxd:51:93: undeclared name not builtin: nogil ``` because `nogil` must be before `except +` * With `cython...
```python import gudhi as gd from gudhi.datasets.generators import points data = points.sphere(n_samples = 30000, ambient_dim = 2, radius = 1000., sample = "random") r = gd.RipsComplex(points=data) from timeit import default_timer...
I completely agree with you. I like the way it is advised in [this c++ tutorial](https://openclassrooms.com/fr/courses/7137751-programmez-en-oriente-objet-avec-c/7532931-gerez-des-erreurs-avec-les-exceptions#/id/r-7533183) (in french). Do you think something equivalent could be added to the [C++ code...
We should also keep in mind the way [cython translates C++ exceptions in python](https://cython.readthedocs.io/en/stable/src/userguide/wrapping_CPlusPlus.html?highlight=io#exceptions)
> Does it not even help if we have tests with the same name in 2 subprojects, or something like that? Anyway, whatever you prefer. From what I read [here](https://cmake.org/cmake/help/latest/guide/tutorial/A%20Basic%20Starting%20Point.html#exercise-1-building-a-basic-project):...
Sorry about that bad merge... I start again a new PR from [96f7508](https://github.com/GUDHI/gudhi-devel/pull/1049/commits/96f7508e6f3b89b4cb13dcdef8e49868c578fd6f) and I close this one. Forget the tensorflow proposal, it will come on a new PR
As explained in [CGAL installation manual](https://doc.cgal.org/latest/Manual/thirdparty.html#title4): > As there is no canonical directory for where to find Boost on Windows, we recommend that you define the environment variable BOOST_ROOT and...
Will the major changes for CGAL 6.0 invalidate the CMake code above? From a CGAL user point of view, if you keep `PACKAGE_VERSION_COMPATIBLE=FALSE` when major version is not the same,...
> Hi @VincentRouvreau @mglisse, > > After a discussion with @sloriot, we decided that we will change that file `CGALConfigVersion.cmake` for `master`/CGAL-6.0. Whatever we decide for the policy in `CGALConfigVersion.cmake`,...