GeometryCollective / boundary-first-flattening

MIT License
764 stars 97 forks source link

Compiling on Ubuntu #66

Open MathSigmamu opened 3 years ago

MathSigmamu commented 3 years ago

This is probably something I am doing stupid but after building the tool on Ubuntu I am getting the following error when using the viewer application.

When I use the command line I get no errors but the vt coordinates in the OBJ are all 0,0,0

CHOLMOD warning: matrix not positive definite. file: ../Supernodal/t_cholmod_super_numeric.c line: 911

rohan-sawhney commented 3 years ago

Hmm, you get this when you launch the application? Or is there a specific mesh you are loading?

MathSigmamu commented 3 years ago

I get it when ever load a mesh, i have tried it with various meshes including those in the input file

DavidJourdan commented 1 year ago

Hi, I have the same issue on Ubuntu as well. This seems to be a bug in CHOLMOD when it is compiled with MKL BLAS and OpenMP (see here: https://github.com/DrTimothyAldenDavis/SuiteSparse/issues/108).

In my case the "not positive definite" message only appears when using the "Supernodal" algorithm. If I replace common.supernodal = CHOLMOD_SUPERNODAL; with common.supernodal = CHOLMOD_SIMPLICIAL; I can manage to solve the system. However, even in that case the resulting parameterization might be of a very poor quality with lots of self-intersections (see the attached screenshot on one of the test models) image

rohan-sawhney commented 1 year ago

With the latest changes, bff seems to run without any problems on Ubuntu 22 as long as suitesparse is installed on your system.