search

Jfortin1 / ComBatHarmonization

Harmonization of multi-site imaging data with ComBat
268 stars 107 forks source link

batch needed to be ordered #22

Closed sreevalsansmenon closed 4 years ago

sreevalsansmenon commented 4 years ago

I had an issue in MATLAB when the batch was unordered. I didn't have data from sites 1 and 2 but had the other 19 sites (from 21 sites in ABCD). But n_batch showed 21 and had some error. I tried using "batchmode(:,~any(batchmod,1))=[];" inline 15 to remove zero columns.

Jfortin1 commented 4 years ago

Thanks for reporting the bug. Will work on it now.

Jfortin1 commented 4 years ago

Should be fixed in https://github.com/Jfortin1/ComBatHarmonization/commit/0e10ddef8364c07c792bb07ac6e57f47dcdaffd7 and https://github.com/Jfortin1/ComBatHarmonization/commit/9bd7015a9974331cd9c8e19d26ec1434e2f2f597