Skip to content

Commit

Permalink
REF: try using less memory in multipletests 'holms' see #1394
Browse files Browse the repository at this point in the history
  • Loading branch information
josef-pkt committed Feb 16, 2014
1 parent 94ff580 commit c86e08b
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion statsmodels/stats/multitest.py
Expand Up @@ -123,10 +123,13 @@ def multipletests(pvals, alpha=0.05, method='hs', returnsorted=False):
----------
'''
import gc
gc.collect()
pvals = np.asarray(pvals)
alphaf = alpha # Notation ?
sortind = np.argsort(pvals)
pvals = pvals[sortind]
#pvals = pvals[sortind]
pvals = np.take(pvals, sortind)
sortrevind = sortind.argsort()
ntests = len(pvals)
alphacSidak = 1 - np.power((1. - alphaf), 1./ntests)
Expand Down Expand Up @@ -168,6 +171,8 @@ def multipletests(pvals, alpha=0.05, method='hs', returnsorted=False):
reject = ~notreject
pvals_corrected_raw = pvals * np.arange(ntests, 0, -1)
pvals_corrected = np.maximum.accumulate(pvals_corrected_raw)
del pvals_corrected_raw
gc.collect()

elif method.lower() in ['sh', 'simes-hochberg']:
alphash = alphaf / np.arange(ntests, 0, -1)
Expand Down

0 comments on commit c86e08b

Please sign in to comment.