spine-generic / data-multi-subject_DO-NOT-USE

Multi-subject data for the Spine Generic project
0 stars 1 forks source link

Fix wrong defacing in some images #19

Closed alexfoias closed 4 years ago

alexfoias commented 4 years ago

This PR fixes the remaining issues with defacing. Fixes #22

alexfoias commented 4 years ago

@jcohenadad This PR is ready for review.

alexfoias commented 4 years ago

Fixed in https://github.com/spine-generic/data-multi-subject/pull/19/commits/0b2399908b823deb0125df70fde74f9547e8da89

Screen Shot 2020-07-31 at 12 09 54 PM

alexfoias commented 4 years ago

The issues signalled here: https://github.com/spine-generic/data-multi-subject/pull/21#issuecomment-667390603

jcohenadad commented 4 years ago

@alexfoias the image shown in https://github.com/spine-generic/data-multi-subject/pull/19#issue-460200227 is not representative of the problem (it is representative of the solution)--

jcohenadad commented 4 years ago
  • sub-cmrra04 & sub-cmrra05 : we received the T2w images like that.

@alexfoias are you sure? sub-cmrra02_T2w.nii.gz , which you were able to fix, is coming from the same site

alexfoias commented 4 years ago

The data that we have from the backup look like this:

Screen Shot 2020-08-03 at 10 30 30 AM Screen Shot 2020-08-03 at 10 30 36 AM

jcohenadad commented 4 years ago

and how does sub-cmrra02_T2w.nii.gz look like? are the other files in the backup not defaced?

alexfoias commented 4 years ago

All the data from cmrr* came defaced.

jcohenadad commented 4 years ago

All the data from cmrr* came defaced.

so how come you were able to "reface" cmrra02?

also see my other question https://github.com/spine-generic/data-multi-subject/pull/19#issuecomment-668058959 are the other files in the backup not defaced?

alexfoias commented 4 years ago

cmrra02 wasn't refaced. The initial file wasn't as chopped as the one that we uploaded before. All the data from cmrr are defaced. Here is a QC of the data from the recovered backup (before defacing of the complete multiSubject).

jcohenadad commented 4 years ago

cmrra02 wasn't refaced

All the data from cmrr* came defaced.

🧐

alexfoias commented 4 years ago

I mixed up things on my fork. I'll close this PR and open a new one with the clean changes.