RivuletStudio / rivuletpy

Robust 3D Neuron Tracing / General 3D tree structure extraction in Python for 3D images powered by the Rivulet2 algorithm. Pain-free Install & use in 5 mins.
BSD 3-Clause "New" or "Revised" License
66 stars 16 forks source link

ModuleNotFoundError: No module named 'msfm' #22

Open meliamne opened 1 year ago

meliamne commented 1 year ago

Dear dev,

I installed the library in a fresh conda env as suggested and pip installed rivuletpy from both pip and github. After the installation, the rtrace command cannot be found by the system. Running either the quicktest or the rtrace app from the github repo reports ModuleNotFoundError: No module named 'msfm'.

Do you have any idea what the issue could be?

Kind regards.

Python 3.9.16

pip list

Package             Version
------------------- ---------
contourpy           1.0.7
cycler              0.11.0
Cython              0.29.33
fonttools           4.38.0
imageio             2.26.0
importlib-resources 5.12.0
kiwisolver          1.4.4
matplotlib          3.7.0
networkx            3.0
nibabel             4.0.2
numpy               1.24.2
packaging           23.0
Pillow              9.4.0
pip                 23.0.1
pyglet              1.5.27
pylibtiff           0.4.4
pyparsing           3.0.9
python-dateutil     2.8.2
PyWavelets          1.4.1
rivuletpy           0.3.0
scikit-fmm          2022.8.15
scikit-image        0.19.3
scipy               1.10.1
setuptools          67.4.0
SimpleITK           2.2.1
six                 1.16.0
tifffile            2023.2.27
tqdm                4.64.1
wheel               0.38.4
zipp                3.15.0
kpeez commented 1 year ago

Hi @meliamne. This is happening because the msfm package isn't being compiled properly for some reason.

The most immediate fix is to remove the import msfm call at the top of the module, and only be imported if using quality=True for R2Tracer.

So this:

    def _fast_marching(self):
        speed = self._make_speed()

        # # Fast Marching
        if self._quality:

            # if not self._silent: print('--MSFM...')
            self._t = msfm.run(speed, self._bimg.copy().astype(
                'int64'), self._soma.centroid, True, True)
        else:
            # if not self._silent: print('--FM...')
            marchmap = np.ones(self._bimg.shape)
            marchmap[self._soma.centroid[0],
                     self._soma.centroid[1], self._soma.centroid[2]] = -1
            self._t = skfmm.travel_time(marchmap, speed, dx=5e-3)

should be changed to this:

  def _fast_marching(self):
        speed = self._make_speed()

        # # Fast Marching
        if self._quality:
            # only import msfm if quality = True
            import msfm 

            # if not self._silent: print('--MSFM...')
            self._t = msfm.run(speed, self._bimg.copy().astype(
                'int64'), self._soma.centroid, True, True)
        else:
            # if not self._silent: print('--FM...')
            marchmap = np.ones(self._bimg.shape)
            marchmap[self._soma.centroid[0],
                     self._soma.centroid[1], self._soma.centroid[2]] = -1
            self._t = skfmm.travel_time(marchmap, speed, dx=5e-3)