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.

Sign up for GitHub

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

Jump to bottom

Replace array-of-array volume representation with ndarrays #21

Closed
tyarkoni opened this issue Aug 6, 2015 · 1 comment · Fixed by #23
Closed

Replace array-of-array volume representation with ndarrays #21

tyarkoni opened this issue Aug 6, 2015 · 1 comment · Fixed by #23
Milestone
first official re...

Comments

@tyarkoni
Copy link
Contributor

tyarkoni commented Aug 6, 2015

See #17 (comment)

We should switch from representing image data as arrays-of-arrays-of-arrays to 1D typed arrays, probably using ndarray. This should speed up performance considerably.
@njvack
Copy link
Contributor

njvack commented Aug 6, 2015

See https://github.com/scijs/ndarray

@tyarkoni tyarkoni added this to the first official release milestone Aug 6, 2015
@tyarkoni tyarkoni mentioned this issue Aug 8, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
first official release
Development

Successfully merging a pull request may close this issue.

Switch to ndarray and collapse layers neurosynth/nsviewer
2 participants
@njvack @tyarkoni