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 function to get test case #235

Merged
merged 3 commits into from
Apr 24, 2017

Conversation

JoostJM
Copy link
Collaborator

@JoostJM JoostJM commented Apr 5, 2017

This function is available in the radiomics namespace and takes the test case name as an argument. Optionally the location of the repository can also be specified, in which case the test files of the repository are used.
If test files cannot be found, the are downloaded from github and stored in a temporary folder.

This function returns a tuple of 2 strings representing the paths to the (downloaded) image and segmentation files.

TODO:

  • Add documentation to this function
  • Test the function
  • Incorporate function in PyRadiomics Notebooks
  • Add logging calls in the function

JoostJM added 3 commits April 5, 2017 17:55
This function is available in the radiomics namespace and takes the test case name as an argument. Optionally the location of the repository can also be specified, in which case the test files of the repository are used.
If test files cannot be found, the are downloaded from github and stored in a temporary folder.

This function returns a tuple of 2 strings representing the paths to the (downloaded) image and segmentation files.
Also add additional logging calls.
Also includes some minor bugfixes and a try-except block around the URLretrieve (can fail when run locally with no internet available).
Add use of the `getTestCase` function in the PyRadiomics example scripts and notebooks.
This also enables the examples to be run if the repository is not available locally.
@JoostJM JoostJM merged commit b820c48 into AIM-Harvard:master Apr 24, 2017
@JoostJM JoostJM deleted the downloadable-testcase branch April 24, 2017 08:36
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.

1 participant