search

yeatmanlab / pyAFQ

Automated Fiber Quantification ... in Python
http://yeatmanlab.github.io/pyAFQ/
BSD 2-Clause "Simplified" License
53 stars 34 forks source link

[FIX] different CST ROI 2 for L/R #1138

Closed 36000 closed 1 month ago

36000 commented 2 months ago
image
arokem commented 2 months ago

OK - just to document what is going on here: in the current software, the inferior ROI for CST contains waypoints in both the left and the right hemispheres. However, because we are excluding streamlines that cross the midline, it makes no sense to have a waypoint in the opposite hemisphere. This PR points the software to new waypoint ROIs that only have voxels in the relevant hemisphere for each CST. Is that correct, @36000? Does that make sense to you as well, @jyeatman?

36000 commented 2 months ago

This is correct

jyeatman commented 1 month ago

Previously we split fibers that crossed rather than excluding them and that’s why the ROIs were in both hemispheres. But this is fine

Jason D. Yeatman, Ph.D.https://profiles.stanford.edu/jason-yeatman Associate Professor Stanford University, Graduate School of Education and Department of Psychology Stanford School of Medicine, Division Developmental Behavioral Pediatrics Brain Development & Education Labhttp://brainandeducation.com/

From: John Kruper @.> Sent: Saturday, May 18, 2024 10:18:00 AM To: yeatmanlab/pyAFQ @.> Cc: Jason Daniel Yeatman @.>; Mention @.> Subject: Re: [yeatmanlab/pyAFQ] [FIX] different CST ROI 2 for L/R (PR #1138)

This is correct

— Reply to this email directly, view it on GitHubhttps://github.com/yeatmanlab/pyAFQ/pull/1138#issuecomment-2118885362, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AANRRSVTOFO4NCQZD6SWOM3ZC6EMRAVCNFSM6AAAAABH5TACAWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJYHA4DKMZWGI. You are receiving this because you were mentioned.Message ID: @.***>