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

Add vector list datatype & add configurable SPACE_DIRECTIONS_TYPE option #157

Merged
merged 36 commits into from
Nov 5, 2024

Conversation

addisonElliott
Copy link
Collaborator

Changes

  • Add int vector list and double vector list datatypes that are lists of Numpy arrays or None.
    • These new types are similar to their int matrix and double matrix counterparts except they are not 2D Numpy matrices.
  • Add nrrd.SPACE_DIRECTIONS_TYPE to enable switching the datatype for the space directions field.
    • Valid options are double matrix or double vector list. The current default is double matrix for backwards compatibility but will be switched to double vector list in the next major release.
    • double vector list is superior over double matrix because it doesn't have the confusing row-of-NaN's representation and it doesn't imply an affine transform by being a matrix.
  • Support row-of-None in addition to row-of-NaN for parse_optional_matrix & format_optional_matrix in addition to new vector list parsing/formatting functions

Fixes #148
Revises #149

@addisonElliott addisonElliott merged commit 62b75f8 into mhe:master Nov 5, 2024
3 checks passed
@addisonElliott addisonElliott deleted the other-improv branch November 5, 2024 10:19
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

Successfully merging this pull request may close these issues.

Confusing representation of "space direction"
1 participant