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

Spatiotemporal decomposition specification for PCA #145

Open
rciric opened this issue Feb 9, 2019 · 3 comments
Open

Spatiotemporal decomposition specification for PCA #145

rciric opened this issue Feb 9, 2019 · 3 comments
Labels
functional MRI help wanted Extra attention is needed

Comments

@rciric
Copy link

rciric commented Feb 9, 2019

This issue regards the BIDS derivatives specification for spatiotemporal decompositions, in particular storage of metadata for components computed by principal component analysis (PCA).

  • For PCA-based decompositions, it would be useful to have metadata specifications for eigenvalues (or singular values) and their derivatives. There was previous discussion about these measures; however, it appears that they might not have made it into the proposal.
    • In the absence of an explicit directive from BIDS, the fmriprep confounds PR implements the following fields (in addition to Method): SingularValue, VarianceExplained, CumulativeVarianceExplained, and Mask (denoting the source mask if more than one mask is used for component extraction).
  • Metadata from components that are not ultimately selected for inclusion in the model matrix can nonetheless provide useful diagnostic information about the decomposition. For instance, in the fmriprep confounds PR, these metadata are used to plot the cumulative variance curve. Since discarded components won’t have corresponding column names in the ~regressors.tsv matrix, it would be useful if there were a standard name for such components to be used when entering their metadata in the decomposition JSON.

In any case, the derivatives spec appears to be frozen for now, so perhaps this can be revisited after it re-opens?

@oesteban oesteban added help wanted Extra attention is needed functional MRI labels Feb 22, 2019
@tsalo
Copy link
Member

tsalo commented May 25, 2020

Was this not included in the extension? I see it in the Google Doc but not in #265.

@effigies
Copy link
Collaborator

@tsalo
Copy link
Member

tsalo commented May 26, 2020

Thank you @effigies! I guess I can circle back once #207 is reopened for input.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
functional MRI help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants