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] Request for Inclusion of "KONG Networks" Option in Nilearn Function "nilearn.datasets.fetch_atlas_schaefer_2018()"" #4074

Closed
1 task done
PuneetDheer opened this issue Oct 20, 2023 · 6 comments
Labels
Atlas Enhancement for feature requests

Comments

@PuneetDheer
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Describe your proposed enhancement in detail.

I was hoping to find the KONG 7 or 17 Network atlas, as referenced in the following source:

[VERSION 3. Schaefer2018_<#ROIs>Parcels_Kong2022_17Networks_order]
https://github.com/ThomasYeoLab/CBIG/tree/master/stable_projects/brain_parcellation/Schaefer2018_LocalGlobal.

However, it seems that this atlas is not available through the nilearn function, as indicated here: https://nilearn.github.io/dev/modules/generated/nilearn.datasets.fetch_atlas_schaefer_2018.html#nilearn.datasets.fetch_atlas_schaefer_2018.

The current functionality only allows for the use of the "yeo_networks" parameter.

I kindly request that you consider adding the KONG 7 or 17 network option, which could be labeled as "kong_networks," similar to the existing "yeo_networks" parameter.

Your consideration in this matter would be greatly appreciated.

Benefits to the change

The inclusion of the "kong_networks" option would enhance the utility of the network atlas by providing access to a broader range of networks, notably, the auditory and language networks/parcels, which are not currently available through the "yeo_networks" parameter.

For reference, you can view the language and auditory parcels here: https://github.com/ThomasYeoLab/CBIG/blob/master/stable_projects/brain_parcellation/Schaefer2018_LocalGlobal/Parcellations/MNI/freeview_lut/Schaefer2018_1000Parcels_Kong2022_17Networks_order.txt.

Your consideration of this valuable addition is sincerely appreciated.

Pseudocode for the new behavior, if applicable

# insert your code below
@PuneetDheer PuneetDheer added the Enhancement for feature requests label Oct 20, 2023
@htwangtw
Copy link
Member

This is a common type of request and I would redirect you to this reply for another thread:

#4039 (comment)

@bthirion
Copy link
Member

Thx for the suggestion.
We already have Schaefer atlas available. https://nilearn.github.io/stable/modules/generated/nilearn.datasets.fetch_atlas_schaefer_2018.html
Isn't it enough ?
Best,

@PuneetDheer
Copy link
Author

Hi bthirion

You are absolutely correct in noting that the "Schaefer atlas" is indeed available. However, it appears that there is no parameter option to select the KONG 7 or 17 network within this atlas. Instead, the atlas only allows the use of the "yeo_networks" parameter.

For your specific needs, I would like to direct your attention to an alternative option: VERSION 3. Schaefer2018_<#ROIs>Parcels_Kong2022_17Networks_order. This version of the Schaefer 2018 atlas provides the capability to access the "Language" and "Auditory" parcels, which are not available when using the "yeo_networks" parameter with the Schaefer atlas. You can find further information on this version on the provided GitHub link.

Additionally, I would like to draw your attention to the image link here, which displays the language and auditory parcels in the Schaefer2018_<#ROIs>Parcels_Kong2022_17Networks_order.

This Kong network option indeed offers access to the specific "Language" and "Auditory" parcels that are not available when using the "yeo_networks" parameter with the Schaefer atlas. For your convenience, you can find more details in the links provided.

Best,

@PuneetDheer
Copy link
Author

Could you please look into it?

@PuneetDheer PuneetDheer reopened this Oct 21, 2023
@bthirion
Copy link
Member

We will discuss this in an upcoming coredev meeting. Best,

@Remi-Gau
Copy link
Collaborator

Was discussed at the last core dev meeting and decided that this is out of scope for Nilearn:

We have updated the documentation to make our policy regarding adding new atlases more obvious (note this is not a change in policy just trying to make it easier to point to) and also pointing to other projects that may accept atlas contribution:

https://nilearn.github.io/dev/development.html#adding-atlases-and-datasets

@Remi-Gau Remi-Gau closed this as not planned Won't fix, can't repro, duplicate, stale Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Atlas Enhancement for feature requests
Projects
None yet
Development

No branches or pull requests

4 participants