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

--combine-runs to work with reho and alff #1396

Open
CogBrainHealthLab opened this issue Feb 10, 2025 · 6 comments
Open

--combine-runs to work with reho and alff #1396

CogBrainHealthLab opened this issue Feb 10, 2025 · 6 comments
Labels
enhancement New feature or request

Comments

@CogBrainHealthLab
Copy link

Currently when the --combine-runs y flag is specified, it outputs a single fMRI volume, however it does not seem to have the same effect for the *alff_boldmap and *reho_boldmap files. Ideally, I would like to have a single reho or alff volume that is concatenated across runs.

@CogBrainHealthLab CogBrainHealthLab added the enhancement New feature or request label Feb 10, 2025
@tsalo
Copy link
Member

tsalo commented Feb 10, 2025

Could you clarify what you mean by ReHo or ALFF being concatenated across runs? Do you mean:

  • A 4D file with one volume for each run
  • An unweighted average across runs
  • A weighted average across runs, with the weighting based on something like number of volumes in the original runs (before or after censoring?)
  • Concatenating the time series and recalculating ALFF and ReHo

@CogBrainHealthLab
Copy link
Author

Definitely not the first option.

would "Concatenating the time series and recalculating ALFF and ReHo" be the most advisable method to combine the ALFF and ReHo volumes from multiple runs into a single volume?

@tsalo
Copy link
Member

tsalo commented Feb 10, 2025

I don't know the literature on combining ALFF and ReHo across runs, so I don't know what folks generally do. To be honest, I think that people should be taking the individual runs' data and using inter-run variability as a measure of subject-level variability instead of concatenating or simply averaging data, but that adds an extra level of complexity to the model.

@mattcieslak
Copy link
Contributor

I think ALFF in particular may not be appropriate for concatenated runs. ALFF is frequency-based, and the discontinuity where data is concatenated may not be ok.

@CogBrainHealthLab
Copy link
Author

what about reho?
Because if its alright to extract the parcellated FC matrix from the concatenated runs, it should make sense to do the same for reho?

@tsalo
Copy link
Member

tsalo commented Feb 13, 2025

It seems okay as far as I can tell, but I don't want to add it without something in the literature to reference, since I don't know the math well enough to say ReHo on concatenated data is valid.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants