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

BUG: Remove extraneous code and widgets from VPAWVisualizeOCT #57

Merged
merged 3 commits into from
Jun 12, 2024

Conversation

Leengit
Copy link
Contributor

@Leengit Leengit commented Jun 10, 2024

Remove parts from VPAWVisualizeOCT that mimicked parts of VPAWVisualize but were not appropriate. Support for the combination of 2-dimensional slices into a 3-dimensional volume can be added to VPAWVisualizeOCT once the combination is supported by the uncbiag/OCTSeg pipeline.

Also, modify the debug-output function summary_repr to optionally be less verbose. Also, format code to respect what ruff check wants.

Although there is more to be done with VPAWVisualizeOCT, I am submitting this pull request so that these changes on my repository fork do not become stale.

Remove parts from VPAWVisualizeOCT that mimicked parts of
VPAWVisualize but were not appropriate.  Support for the combination
of 2-dimensional slices into a 3-dimensional volume can be added to
VPAWVisualizeOCT once the combination is supported by the
uncbiag/OCTSeg pipeline.
@Leengit Leengit added the bug Something isn't working label Jun 10, 2024
@Leengit Leengit self-assigned this Jun 10, 2024
Copy link
Contributor

@ebrahimebrahim ebrahimebrahim left a comment

Choose a reason for hiding this comment

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

All looks good

@Leengit Leengit merged commit 53282e7 into KitwareMedical:main Jun 12, 2024
1 check passed
@Leengit Leengit deleted the visualize_oct branch June 12, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants