Closed valosekj closed 4 years ago
@valosekj some feedback:
sub-juntendoAchieva
for GM-CSA, it is already excluded: https://github.com/spine-generic/data-single-subject/blob/master/exclude.yml#L2; however, we need to change sub-juntendoAchieva --> juntendoAchieva (in order to display the site in the graph)sub-perform
: looks good to mesub-glen
: agreed, it needs correctionsub-tokyoSkyra
: agreed, we could correctsub-unf
: looks good to mehow are mgh and juntendo750w? they look like outliers.
1.
- regarding
sub-juntendoAchieva
for GM-CSA, it is already excluded: https://github.com/spine-generic/data-single-subject/blob/master/exclude.yml#L2; however, we need to change sub-juntendoAchieva --> juntendoAchieva (in order to display the site in the graph)
Okay, should I open PR with updated exclude.yml
file?
2.
how are mgh and juntendo750w? they look like outliers.
sub-mgh_T2star_rms.nii.gz
- looks good to me
sub-juntendo750w_T2star_rms.nii.gz
- also not so bad for me
3.
sub-perform
: looks good to mesub-glen
: agreed, it needs correctionsub-tokyoSkyra
: agreed, we could correctsub-unf
: looks good to me
Thanks, it is not easy to find subjective threshold, what is sufficient and what needs corrections.
Okay, should I open PR with updated exclude.yml file?
yes please
Fixed
I found following processing issues in r20200826 release.
GM to correct:
sub-juntendoAchieva_T2star_rms.nii.gz
sub-perform_T2star_rms.nii.gz
sub-glen_T2star_rms.nii.gz
sub-tokyoSkyra_T2star_rms.nii.gz
sub-unf_T2star_rms.nii.gz