jerdra / SDC-BIDS-fMRI_old

Scientific-Compute Working Group Workshop on performing analysis of neuroimaging data in python
3 stars 10 forks source link

Ep 05 #22

Open josephmje opened 5 years ago

josephmje commented 5 years ago

Pandas concepts to move to intro dataset notebook

josephmje commented 5 years ago

Move teaching pandas to "intro to dataset" notebook. I'm working on an ipywidget for visualizing time series changes. Will submit a WIP PR for this soon.

jerdra commented 5 years ago

For explaining what a TR is, this is something that should be explained in the Nibabel section or the intro to nilearn section. I'll update accordingly to deal with 4D images and how we can think about them.

josephmje commented 5 years ago

maybe nilearn since i've only got T1s in nibabel

jerdra commented 5 years ago

sure, it does feel like it fits more naturally in the nibabel section (after talking about 3D arrays before visualization), but I think reducing the density of intro content would be good.

josephmje commented 5 years ago

ahhh yea i see what you mean. i do mention the 4th dimension as time when describing the shape of the array. maybe we can do a call back to that part

jerdra commented 5 years ago

done implementing in nilearn notebook, i just used the same fig and extended it to the 4th dimension, think it should be okay!

jerdra commented 5 years ago

checking off python-language because that's data carpentry specific..

jerdra commented 5 years ago

27 contains updates to better explain confound_df columns

jerdra commented 5 years ago

27 has small explanation of temporal derivatives using X position confound and "X velocity" as example

jerdra commented 5 years ago

Re: "Applying confound regression", I think we can leave it up there and just skip it in the notebook

jerdra commented 5 years ago

Re: "Exercises", let's do the exercise at the very beginning of the functional connectivity section? Thoughts?

josephmje commented 5 years ago

Don't think I have row indexing in pandas yet. Should that also be added to the dataset notebook?

josephmje commented 5 years ago

Yup, I like the idea of applying it to the FC notebook. Would be a nice way to recap everything.

jerdra commented 5 years ago

if there's a way to make it part of working with the data that would be great, if not i don't mind explaining it in the signal cleaning section. Just let me know so I can work on it asap :+1:

Yeah ideally! But there's also the question of whether we just want to jump to NiftisLabelsMasker instead of using image.clean again? We might have to remove first bit for timing

josephmje commented 5 years ago

I think I can make row indexing work!

True, should we keep image.clean in there but then skip to NiftiLabelsMasker if we notice we're running low on time?

jerdra commented 5 years ago

Sure I think that's reasonable!

jerdra commented 5 years ago

Need to use pybids here, will make a PR in a bit

jerdra commented 5 years ago

done