-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add initial set of metrics #19
Conversation
out_samplerate : :obj:`float` | ||
Sampling rate for the output time series in seconds. | ||
Corresponds to TR in fMRI data. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to include downsampling in the individual metric functions, or have it in a separate step?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm thinking that downsampling should be done somewhere else TBH.
Move metric call function to metrics.utils
Add Cesar's RETROICOR code
🚀 PR was released in |
---------- | ||
.. [1] C. Chang & G. H. Glover, "Relationship between respiration, | ||
end-tidal CO2, and BOLD signals in resting-state fMRI," Neuroimage, | ||
issue 4, vol. 47, pp. 1381-1393, 2009. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A bit of a nit-pick, but I think RV was described in "Influence of heart rate on the BOLD signal: the cardiac response function" also by Chang and Glover in 2009. It may also be in this paper, but I didn't see it from a skim.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should handle it in a new PR
Closes #2 and closes #3 and closes #7.
Proposed Changes
Change Type
bugfix
(+0.0.1)minor
(+0.1.0)major
(+1.0.0)refactoring
(no version update)test
(no version update)infrastructure
(no version update)documentation
(no version update)other
Checklist before review