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

Bitwise data / groups #3

Open
Lestropie opened this issue Nov 16, 2020 · 2 comments
Open

Bitwise data / groups #3

Lestropie opened this issue Nov 16, 2020 · 2 comments

Comments

@Lestropie
Copy link

  • Currently there is no example showcasing the use of bitwise data. Technically this datatype is applicable to both DPP and DPS.

  • It is also technically feasible for bitwise DPS to be utilised as group definition, rather than a list of indices. This would only reduce requisite space for groups that select more than 1/32 of streamlines, and unlike a list of indices it would mean that the number of elements cannot be inferred without reading the data, but it's nevertheless worth discussing the prospect, and explicitly rejecting if that's the consensus.

@frheault
Copy link
Collaborator

That's an interesting point that brings back an idea from the original thread (except it was a DPS in uint8 and required a LUT for meaning).
We just have to mention what would be the extension for binary data type? just .bool?

It is a simple idea that allows for quite a lot of liberty. I don't see how it could be misused, I would add it immediately unless anyone has a counter-argument.

@Lestropie
Copy link
Author

We just have to mention what would be the extension for binary data type? just .bool?

I'd probably advocate for .bit rather than .bool. Over and above being consistent with existing MRtrix3 parlance, I think it's a more direct reference to storage, whereas "boolean" to me conveys either an internal interpretation of data or the logic surrounding such.

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