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 parameter file support #36

Merged
merged 3 commits into from Jan 9, 2018
Merged

Add parameter file support #36

merged 3 commits into from Jan 9, 2018

Conversation

JoostJM
Copy link
Contributor

@JoostJM JoostJM commented Jan 9, 2018

Update the user interface (SlicerRadiomicsWidget) to optionally extract features using a PyRadiomics Parameter File.

  • Group the 3 different types of customization (Feature Class, filter and settings) together into 1 groupbox
  • Add radiobuttons to toggle between manual customization (using UI elements) and parameter file customization (using a parameter file)
  • On toggle, hide to non-selected group box
  • When in parameter file customization mode, disable the Apply button so long as the paramter file path element does not contain an existing file
  • Split the setup function into separate functions per section to improve readability

Update the logic (SlicerRadiomicsLogic) to support the parameter file

  • Add a second run function: runWithParameterFile
  • Instantiate the extractor in the respective run functions
  • Decode the labelNode and Segementation node in the calculateFeatures Function

Add a test running SlicerRadiomicsLogic with an example parameters file

Update the user interface (SlicerRadiomicsWidget) to optionally extract features using a PyRadiomics Parameter File.

- Group the 3 different types of customization (Feature Class, filter and settings) together into 1 groupbox
- Add radiobuttons to toggle between manual customization (using UI elements) and parameter file customization (using a parameter file)
- On toggle, hide to non-selected group box
- When in parameter file customization mode, disable the `Apply` button so long as the paramter file path element does not contain an existing file
- Split the setup function into separate functions per section to improve readability

Update the logic (SlicerRadiomicsLogic) to support the parameter file

- Add a second run function: `runWithParameterFile`
- Instantiate the extractor in the respective run functions
- Decode the labelNode and Segementation node in the `calculateFeatures` Function
Add a test which runs the SlicerRadiomics logic using an example parameters file downloaded from the github repository (included in the test data).
@JoostJM JoostJM merged commit 258d1e9 into master Jan 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant