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

DOC: Fix example to "Access voxels of a 4D volume as a single numpy array #7143

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jcfr
Copy link
Member

@jcfr jcfr commented Aug 1, 2023

This was created based of changes originally contributed through:

@jcfr jcfr requested a review from lassoan August 1, 2023 06:35
@jcfr jcfr enabled auto-merge (rebase) August 1, 2023 06:35
@@ -21,13 +21,13 @@ Get all voxels of a 4D volume (3D volume sequence) as a numpy array called `voxe
```python
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Description is inconsistent with example.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Following up developer meeting:

  • Add another examples for 5D
  • Ideally vtkTeemNRRD reader could be fixed to support 5 components

Related to:

@jcfr jcfr added the Type: Documentation Issues regarding documentation label Aug 17, 2023
@jcfr
Copy link
Member Author

jcfr commented Mar 6, 2024

@lassoan When you have a chance, Ii would be curious to get your perspective on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Issues regarding documentation
Development

Successfully merging this pull request may close these issues.

None yet

2 participants