-
Notifications
You must be signed in to change notification settings - Fork 1
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
Use with 10 um atlas? #43
Comments
This is somewhat embarassing, I thought I shared the format somewhere, but that isn't the case.
The code reads a NIfTI volume (which is gzip-d) coupled with an ITK label file. Parsing the NIfTI is skipped though, it assumes a 352-byte header, 456x528x320 voxels resolution (and it's RAS), and 32-bit little endian elements. |
I wonder how small we can get the 10um annotations. On the Allen website they are only 26MB. What in your opinion @Tevemadar is an acceptable size? |
@PolarBean This RLE hack gained 20-25% for the 25um segmentations (4-ish MB to 3-ish MB). WHS is similar (3-4 MB-s to 2-2.5 MB-s). Which is okay for a cheap, 10-line decompressor, but not astonishing. @tmchartrand ehm, when writing this comment, I looked into the code and noticed that the decompressor for the variant with "encodeN" doesn't actually exist. Please let me know if you need it. |
This is all very helpful - I'm not sure yet whether I'll put time into this, will probably spend a bit more time exploring the tool as it is first, but I'll let you know if/when I try anything new. |
"encodeN" was meant for the run-lengths only. It may gain something in the empty spaces above and below the curved parts, as at 10um resolution there certainly are a lot of consecutive 0-voxels in those parts. |
This is so smooth! |
Hi, I'm curious if you've considered using this with a 10um version of the ABA 2017 atlas - would the performance issues likely be prohibitive? If it seems plausible but just hasn't been tried, do you have any documentation of the atlas file format (or even better, conversion scripts) in case I wanted to test it out?
The text was updated successfully, but these errors were encountered: