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

Change memory mapped array sampling scheme to match NeuroVault #81

Open
tyarkoni opened this issue Apr 29, 2015 · 4 comments
Open

Change memory mapped array sampling scheme to match NeuroVault #81

tyarkoni opened this issue Apr 29, 2015 · 4 comments

Comments

@tyarkoni
Copy link
Contributor

Currently the memory mapped numpy arrays used in decoding sample ~20k random voxels from the whole brain. NeuroVault resamples images to isotropic 4 mm voxels, which gives ~30k voxels. It would probably speed up processing a bit to use the same scheme in Neurosynth, plus results might be slightly more representative using resampled voxels rather than random ones.

@vsoch
Copy link

vsoch commented Apr 29, 2015

+1

@vsoch
Copy link

vsoch commented Apr 29, 2015

priority:awesome

@tyarkoni
Copy link
Contributor Author

Well, realistically, this might take us from ~0.5s per decoding to ~0.3s, so I'm not sure it's worth upping the priority. But next time I'm mucking around with that part of the code, I'll add this. :)

@vsoch
Copy link

vsoch commented Apr 29, 2015

The level of awesomeness is not impacted by being low priority! :O)

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

No branches or pull requests

2 participants