-
Notifications
You must be signed in to change notification settings - Fork 581
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
Consolidating functional connectivity examples: merge 8.4.1 and 8.4.7 #1986
Comments
As recommended by @KamalakerDadi , would also be useful to merge 8.4.5 (Extracting brain signal from spheres) and 8.4.8 (Extract signals on spheres from an atlas and plot a connectome) into one notebook with both spheres and a full atlas. So for reference:
|
This looks like it was closed with #2013. |
Should we open a new PR specifically for that or do you want to handle it in #1890, @KamalakerDadi ? |
I think you followed it better than me. |
Opened #2183 , if you'd like to review @KamalakerDadi ! 😸 |
Thanks! |
Some of the current nilearn examples include some redundancy that creates unnecessary burden for test run-times.
An example of this would be 8.4.1 (Group analysis of resting-state fMRI with ICA: CanICA) and 8.4.7 (Dictionary Learning and ICA for doing group analysis of resting-state fMRI), which contain a good deal of redundancy.
The goal here would be to create a single example that combines the non-redundant elements from both 8.4.1 and 8.4.7., and perhaps modifying the didactic structure such that it might better reflect the goals and thought-process of an end user.
The text was updated successfully, but these errors were encountered: