Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add voxel cli #457

Merged
merged 4 commits into from
Jan 31, 2019
Merged

Add voxel cli #457

merged 4 commits into from
Jan 31, 2019

Conversation

JoostJM
Copy link
Collaborator

@JoostJM JoostJM commented Jan 31, 2019

Update command line entry point to allow either segment- or voxel-based extraction.
Update documentation to detail usage of PyRadiomics in voxel-based extraction.

Add a new switch in PyRadiomics commandline entrypoint: `--mode`, `-m`, which takes either `segment` (default) or `voxel`.

Add a new switch in PyRadiomics commandline entrypoint: `--out-dir`, `-od`, which specifies a directory (created if not exists) where featuremaps (mode `voxel`) or intermediate files (mode `segment`) are stored. If not specified, featuremaps are stored in the current working directory and intermediate files are not stored.

Refactor the commandline parsing and running of PyRadiomics.
Allow adding diagnositic info to the output when extracting voxel-based features.

After extracting features in a case, results are split into featuremaps, which are stored as SimpleITK images and non-features, which are processed as the output of segment-based extraction.
@JoostJM JoostJM merged commit b92bd22 into AIM-Harvard:master Jan 31, 2019
@JoostJM JoostJM deleted the add-voxel-cli branch February 20, 2019 14:21
JoostJM added a commit that referenced this pull request Mar 13, 2019
Because of changes made in #457 (more specifically, commit d90aa7f), diagnostic information is now also provided if voxel-based extraction is enabled. This requires type checking on the output in `helloVoxel.py`, as diagnostic features cannot be stored as a SimpleITK image.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant