KitwareMedical / KitwareMedical/lungair-desktop-application
consider using slicer's built in numpy interfacing options
- Dominant language
- Python
- Stars
- 7
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Description
Look into using `slicer.util.updateVolumeFromArray` and `slicer.util.updateSegmentBinaryLabelmapFromArray` to replace the current low-level approach [here](https://github.com/KitwareMedical/lungair-desktop-application/blob/master/Modules/Scripted/Home/HomeLib/image_utils.py)
[slicer.util docs](https://slicer.readthedocs.io/en/latest/developer_guide/slicer.html)
Contributor guide
Research direction
Start in Modules/Scripted/Home/HomeLib/image_utils.py and compare the current low-level array-to-volume handling with slicer.util.updateVolumeFromArray and slicer.util.updateSegmentBinaryLabelmapFromArray in the linked Slicer documentation. The work is done when the applicable low-level approach is replaced by these built-in interfaces without changing the intended volume or labelmap updates.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- numpy, python
- Domain
- desktop
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100