search

bast / tms-location

Code to visualize and locate points for transcranial magnetic stimulation (TMS).
GNU Affero General Public License v3.0
0 stars 0 forks source link

Error running tms-location.sif #3

Closed peraslaksen closed 1 day ago

peraslaksen commented 1 month ago

./tms-location.sif /home/per/bast/328 Traceback (most recent call last): File "/software/app.py", line 101, in solver = create_solver(points, vertices) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/software/geodesic.py", line 8, in create_solver return pp3d.EdgeFlipGeodesicSolver(vertices, faces) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/software/venv/lib/python3.11/site-packages/potpourri3d/mesh.py", line 64, in init self.bound_solver = pp3db.EdgeFlipGeodesicsManager(V, F) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ RuntimeError: GC_SAFETY_ASSERT FAILURE from /project/deps/geometry-central/src/surface/manifold_surface_mesh.cpp:158 - vertex 4306 appears in more than one boundary loop

bast commented 3 weeks ago

The meshes might have been generated by an older version of https://github.com/bast/MRI-extract-surfaces/. But if not, then this is a real problem that I know how to fix in the pre-processing step (but I thought I fixed this already).

I will add version info to the mesh folder so that we always know which version generated them.

peraslaksen commented 3 weeks ago

Hi, will check the laptop at home, at the workstation the output folder contains all the 1001-1010 files in addition to the outside only. Nonetheless, I will update all to the newest,

Thanks!

P

From: Radovan Bast @.> Sent: Friday, June 14, 2024 10:16 To: bast/tms-location @.> Cc: Per M. Aslaksen @.>; Author @.> Subject: Re: [bast/tms-location] Error running tms-location.sif (Issue #3)

The meshes might have been generated by an older version of https://github.com/bast/MRI-extract-surfaces/. But if not, then this is a real problem that I know how to fix in the pre-processing step (but I thought I fixed this already).

I will add version info to the mesh folder so that we always know which version generated them.

— Reply to this email directly, view it on GitHubhttps://github.com/bast/tms-location/issues/3#issuecomment-2167500498, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJAENBRWOQUOUCGE5WDY6J3ZHKRFJAVCNFSM6AAAAABI5P6B4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRXGUYDANBZHA. You are receiving this because you authored the thread.Message ID: @.***>

bast commented 1 day ago

I will close this one but let's reopen if the issue is still here. Now the mesh directory also contains version information so then I can more easily correlate it with code changes.