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

Loading ocontours added #2

Merged
merged 2 commits into from
Apr 14, 2018
Merged

Loading ocontours added #2

merged 2 commits into from
Apr 14, 2018

Conversation

tomasprinda
Copy link
Owner

@tomasprinda tomasprinda commented Apr 14, 2018

core.py

  • I created class ContourMask modified class AnnotatedScan so that it holds 2 instances of ContourMask. One for i-contour -> imask, the other o-contour ->omask.

parse.py

  • I created get_contour_mask() which takes care of creating ContourMask() objects
  • I created load_annotated_scan() which takes care of loading one single AnnotatedScan() which simplified method load_patient_scans()

I adjusted tests to reflect the changes

Wrong data!!!
I found out o-contours for patient with DICOM_ID=SCD0000501 are shifted so I am skipping loading data for this patient. See experiments/test_imask_inside_omask_without_skipping/

To make sure to catch these kind of error I created test_imask_inside_omask()

5

@tomasprinda tomasprinda merged commit d1226ac into master Apr 14, 2018
@tomasprinda tomasprinda deleted the ocontours_load branch April 14, 2018 14:55
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.

None yet

1 participant