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

Remove python equivalents for c functions #373

Merged

Conversation

JoostJM
Copy link
Collaborator

@JoostJM JoostJM commented Apr 11, 2018

Remove Python Texture matrix calculation:

  • GLCM
  • GLRLM
  • GLSZM
  • GLDM
  • NGTDM

Remove Python surface area calculation
Remove Python GenerateAngles()
Remove parameters associated with enabling/disabling C Extensions
Remove Python calculation from test_cmatrices.py. Instead, compare to baseline files (binary numpy array files).
Additionally, rename the script to test_matrices.py.

Update and move the script to add a new baseline to the tests folder, so that it can utilize the testUtils module
during generation of a new baseline. The updates now include the generation of a baseline matrix file if the class
contains the appropriate function (_calculateCMatrix).

Update warning messages displayed when C extensions fail to load.
Update examples.
Update documentation.

cc @Radiomics/developers

Remove Python calculation from `test_cmatrices.py`. Instead, compare to baseline files (binary numpy array files).
Additionally, rename the script to `test_matrices.py`.

Update and move the script to add a new baseline to the tests folder, so that it can utilize the testUtils module
during generation of a new baseline. The updates now include the generation of a baseline matrix file if the class
contains the appropriate function (`_calculateCMatrix`).
Remove Python Texture matrix calculation:
 - GLCM
 - GLRLM
 - GLSZM
 - GLDM
 - NGTDM
Remove Python surface area calculation
Remove Python `GenerateAngles()`
Remove parameters associated with enabling/disabling C Extensions
Update warning messages displayed when C extensions fail to load.
Update examples.
Update documentation.
@JoostJM JoostJM force-pushed the remove-python-equivalents-for-c-functions branch from 93dc12c to 0c80bdb Compare April 11, 2018 14:20
@JoostJM JoostJM merged commit 1b0eb74 into AIM-Harvard:master May 31, 2018
@JoostJM JoostJM deleted the remove-python-equivalents-for-c-functions branch June 4, 2018 11:21
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