KitwareMedical / KitwareMedical/lungair-desktop-application

consider using slicer's built in numpy interfacing options

Open
#9 0 comments 0 reactions 0 assignees View on GitHub
enhancement low priority
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.