search

DCBIA-OrthoLab / SlicerDentalModelSeg

This extension aims to provide a GUI for a deep-learning automated teeth segmentation tool that we developed at the University of North Carolina in Chapel Hill in collaboration with the University of Michigan in Ann Arbor.
Apache License 2.0
44 stars 10 forks source link

Python package question #5

Closed Connor-Bowley closed 2 years ago

Connor-Bowley commented 2 years ago

If I recall, you opted for using the system python environment so you could get newer packages than were available in Python 3.6. With the update of Slicer's (not SlicerSALT's yet) Python to 3.9.10, can you get everything you need into the Slicer python environment?

MathieuLeclercq commented 2 years ago

I am working on Python 3.9.7 so it should be possible to get everything I need into the Slicer Python environment when it updates to 3.9.10.

Connor-Bowley commented 2 years ago

The Slicer mainline has update to Python 3.9.10, and SlicerSALT will at some point. After talking with @bpaniagua we think you should update your extension to be compatible with the mainline slicer (i.e. get everything into the Slicer Python environment and build against the head of Slicer). Once that is done, we can merge the extension index PR. Your extension won't be able to get put into SlicerSALT until SlicerSALT updates to the head of Slicer as well, but we need to do that anyway.

So long story short, you are good to start on this.

MathieuLeclercq commented 2 years ago

Thank you. I am going to be working on that.

Connor-Bowley commented 2 years ago

How is this coming?

MathieuLeclercq commented 2 years ago

I managed to install all required dependencies. I Still need to fix some errors due to the installation of a newer version of PyTorch that causes some minor incompatibilities. I will get back to you when everything is done.

MathieuLeclercq commented 2 years ago

The module is now fully working on PythonSlicer.