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

[ENH] cluster threshold for surface GLM #4326

Open
1 task done
mszinte opened this issue Mar 19, 2024 · 1 comment
Open
1 task done

[ENH] cluster threshold for surface GLM #4326

mszinte opened this issue Mar 19, 2024 · 1 comment
Labels
Enhancement for feature requests GLM Issues/PRs related to the nilearn.glm module. Surface Related to surface data or surface analysis.

Comments

@mszinte
Copy link

mszinte commented Mar 19, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Describe your proposed enhancement in detail.

As I use these days mostly (if not only) surface analysis of fMRI data, I use to the surface GLM analysis of nilearn, however I couldn't find the cluster_threshold method available for volume analysis.
The idea of cluster analysis for surface would be to discard isolated vertices from the GLM output in order to generate thresholded maps with small cluster of vertices removed following a cluster_threshold argument.
Here is the volume based equivalent:

def threshold_stats_img(

After early discussion I feel that for this improvement (@Remi-Gau @bthirion, @ulascombes) , we would need first:

  • to associate each vertex to an area
  • define connection component of each vertex
  • set rules of connection of a vertices (maybe geodesic distances)

Benefits to the change

This analysis is to my opinion an important missing step for the basic GLM surface analysis of nilearn, it will benefit to the community of users who moved toward fMRI surface analyses.

Pseudocode for the new behavior, if applicable

# insert your code below
@mszinte mszinte added the Enhancement for feature requests label Mar 19, 2024
@bthirion
Copy link
Member

Well, you don't want to have to compute geodesic distances to obtain connected clusters. You simply need to use the topology of the mesh (corresponding to the face definition).
But I agree that this would be useful.

@Remi-Gau Remi-Gau added GLM Issues/PRs related to the nilearn.glm module. Surface Related to surface data or surface analysis. labels Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement for feature requests GLM Issues/PRs related to the nilearn.glm module. Surface Related to surface data or surface analysis.
Projects
None yet
Development

No branches or pull requests

3 participants