Improve integration with autoscoper

BrownBiomechanics / SlicerAutoscoperM

This 3D Slicer extension enables users to perform image registration.

https://autoscoperm.slicer.org

MIT License

0 stars 3 forks source link

Improve integration with autoscoper #7

Closed jcfr closed 1 year ago

jcfr commented 1 year ago

Rename module from AutoscoperConnect to AutoscoperM
Update icon for AutoscoperM extension & module
Improve support for starting/stopping autoscoper
- Move "Launch" button to the top
- Add "Close" button for closing the process
- Simplify the logic expecting the executable to be in the PATH
Refactor Autoscoper communication using PyAutoscoper
Remove obsolete AutoscoperMockServer module
Remove unused sample data registration and testing code

Before	After

Context

This pull-request is based of:

https://github.com/BrownBiomechanics/SlicerAutoscoperM/pull/4

NicerNewerCar commented 1 year ago

@jcfr I'm getting this error when I try to configure the project with this command cmake .. -DSlicer_DIR="C:/D/S4R/Slicer-build" -DQt5_DIR="C:/Qt/5.15.2/msvc2019_64/lib/cmake/Qt5" -DSlicerAutoscoperM_SUPERBUILD=ON

CMake Error at C:/D/S4/CMake/SlicerExtensionCPack.cmake:174 (message):
  SlicerAutoscoperM: Variable CPACK_INSTALL_CMAKE_PROJECTS is expected to be
  set.
Call Stack (most recent call first):
  CMakeLists.txt:27 (include)

Not sure what I need to set this to, I didn't have it set to anything before.

Update:

It appears my version of slicer may be broken, I am rebuilding now.

jcfr commented 1 year ago

Notes:

The option -DSlicerAutoscoperM_SUPERBUILD=ON should not be set here, this will be integrated in a follow up PR. If you already have an extension build, set to OFF. Also note that this option does not need to be explicitly set when building extension, it will default to the expected value.
To ensure autoscoper is found, call the following before run the SlicerWithSlicerAutoscoperM.exe launcher
```
set PATH="C:\path\to\Autoscoper-build\Autoscoper-build\bin\Release;%PATH%"
```