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

Allow P-value cluster-forming thresholds with pseudo-t #44

Open
nicholst opened this issue Apr 3, 2017 · 3 comments
Open

Allow P-value cluster-forming thresholds with pseudo-t #44

nicholst opened this issue Apr 3, 2017 · 3 comments

Comments

@nicholst
Copy link
Collaborator

nicholst commented Apr 3, 2017

Because it is such a pain, we might as well offer to convert P-values into corresponding T/F statistics based on nominal degrees of freedom when the Pseudo-t statistic is used. Variance smoothing messes up the DF, and hence these statistics won't have the exact same interpretation as usual... however, there is no known mapping between uncorrected P-values and pseudo-t values, and hence it leaves users in a muddle.

In FSL's randomise, we don't worry about this.

We should change our behavior, and provide the P-to-T/F service for users, simply issuing a warning that the conversion is approximate due to unknown effective degrees of freedom.

Relevant code starts here: https://github.com/nicholst/SnPM-devel/blob/master/snpm_pp.m#L518

@cmaumet
Copy link
Member

cmaumet commented Apr 4, 2017

@nicholst
Copy link
Collaborator Author

nicholst commented Apr 4, 2017

Oh... oops... yes.

@nicholst nicholst closed this as completed Apr 4, 2017
@nicholst nicholst reopened this Apr 4, 2017
@nicholst
Copy link
Collaborator Author

nicholst commented Apr 4, 2017

No.. but wait... that code assumes that the statistic image is T... not F... current code won't do something sensible with F statistic.

ACTION: Need to revise code to allow sensible clustering forming threshold (from P-value) when STAT == 'F'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants