Function to analyze lesions #1351 #1439
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 30 commits
April 19, 2017 13:54
Angle correction
peristeri
previously requested changes
Sep 6, 2017
scripts/sct_analyze_lesion.py
Outdated
| @@ -232,6 +232,8 @@ def _measure_volume(self, im_data, p_lst, idx): | |||
| printv(' Volume : ' + str(round(vol_tot_cur, 2)) + ' mm^3', self.verbose, type='info') | |||
|
|
|||
| def _measure_length(self, im_data, p_lst, idx): | |||
| print len(self.angles) | |||
There was a problem hiding this comment.
Are these print statements required? If so, you can use sct_utils.printv instead.
There was a problem hiding this comment.
Removed! Is it now correct?
peristeri
added
feature
|
@charleygros was there a test file part of this pull request? |
|
@benjamindeleener @jcohenadad @charleygros is the design discussion on analysing lesions completed? |
jcohenadad
approved these changes
Sep 8, 2017
jcohenadad
added a commit
that referenced
this pull request
Sep 13, 2017
* jca-release: DOC: CHANGES: updated for 3.0.8 Function to analyze lesions #1351 (#1439) BUG: propseg: added try/except for QC report (issue #1456) (#1457) Conversion issue for float32 images with large dynamic (#1450) Now possible to input single label at disc (instead of mid-body) (#1451) Various improvements (#1448) BUG: extract_metric: moved diagonalization for map and ml cases only (issue 1445) (#1446) Now using N-1 instead of N as denominator for computing the STD. (#1441) Updated the list of articles citing SCT. (#1437) Properly exit the installation process if error occurs DEV: fixed bug on centerline when referencing to the PMJ
jcohenadad
added a commit
that referenced
this pull request
Oct 9, 2017
* master: (31 commits) Introduced the zoom functionality to the anatomical canvas (#1477) Poq issue 1409 log (#1411) The minimal QT dialog (#1379) Update README.md download_data: forgot to add pmj_models in the -d flag INST: added PMJ models to OSF Remove models Add models to data folder Update README.md DEV: outputs continuous pixel coordinates of centerline DEV: improvements and fixing bugs DEV: added option to select the gap between points NEW: function for generating a centerline using a few points on a viewer Documentation and Usage BUG: download_data: updated binaries to fix issue #1464 (#1465) BUG: image/keep-vol: wrong argument passed Function to analyze lesions #1351 (#1439) BUG: propseg: added try/except for QC report (issue #1456) (#1457) Conversion issue for float32 images with large dynamic (#1450) DEV: new method to get interpolated coordinate along centerline ... # Conflicts: # scripts/sct_pipeline.py # scripts/sct_testing.py # testing/test_sct_propseg.py
jcohenadad
added a commit
that referenced
this pull request
Dec 18, 2019
* jca-release: DOC: CHANGES: updated for 3.0.8 Function to analyze lesions #1351 (#1439) BUG: propseg: added try/except for QC report (issue #1456) (#1457) Conversion issue for float32 images with large dynamic (#1450) Now possible to input single label at disc (instead of mid-body) (#1451) Various improvements (#1448) BUG: extract_metric: moved diagonalization for map and ml cases only (issue 1445) (#1446) Now using N-1 instead of N as denominator for computing the STD. (#1441) Updated the list of articles citing SCT. (#1437) Properly exit the installation process if error occurs DEV: fixed bug on centerline when referencing to the PMJ Former-commit-id: 16dd07d
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Requirements
TODO in SourceForge?
Applicable Issues
Implements #1351