search

fangq / brain2mesh

Brain2Mesh - a one-liner for high-quality brain mesh generation
http://mcx.space/brain2mesh
Other
34 stars 14 forks source link

Error due to old `tetgen` binary #15

Closed rcassani closed 2 years ago

rcassani commented 2 years ago

Not sure if the issues belongs here or in iso2mesh

In some places brain2mesh.m (example) calls meshabox.m which in turns calls surf2mesh.m without method thus the default 'tetgen' is used. This causes an error as the old binary is used.

The error is solved by replacing the old binary, tetgen.mexa64, with a symbolic link to tetgen1.5.mexa64

fangq commented 2 years ago

For meshabox, I do not believe using the old tetgen (1.4.3) should cause any issue. In fact, tetgen 1.4.3 also works for brain2mesh steps, with the exception that it does not automatically label individual regions. That's the only place where tetgen1.5 is more appropriate.

what error did you see when calling meshabox?

rcassani commented 2 years ago

@fangq, thanks for the quick reply. Indeed the problem was not caused by the old binary, but to the fact that binary did not have execute permissions when downloaded directly from the repo.