Skip to content

Conversation

@mariamonzon
Copy link

include test for detection by @mariamonzon in mariamonzon#2
What's Changed
Update numpy ans isotropic voluumes by @mariamonzon in mariamonzon#1
make spinNet compatible with different pixel spacing [px,py]
validate detection and polygon coords are within image size
include test for detection by @mariamonzon in mariamonzon#2
include how to install in README by @mariamonzon in mariamonzon#3
New Contributors
@mariamonzon
Full Changelog: https://github.com/mariamonzon/SpineNet/commits/v.2.0.3

Copy link
Owner

Choose a reason for hiding this comment

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

This python version doesn't match that in the .readthedocs.yaml file (3.9)

) -> Tuple[List[List[torch.Tensor]], List[List[Dict]]]:
"""
Exhaustively split 3D scan volume into resized, normalized patches for detection.
Ensures consistency with split_into_patches_exhaustive when px=py.
Copy link
Owner

Choose a reason for hiding this comment

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

px, py = x and y pixel spacing? If the split_into_patches_exhaustive is redundant now we should remove it.

Copy link
Author

Choose a reason for hiding this comment

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

is redundant but I did not want to completely remove because I was still in testing phase

Copy link
Owner

Choose a reason for hiding this comment

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

Ah ok - guess it can go now then - it's in the history so no worries if we need to bring it back.

Copy link
Owner

Choose a reason for hiding this comment

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

Not sure what this submodule is, can we remove from MR?

Copy link
Author

Choose a reason for hiding this comment

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

Oh this is a mistake, I will!

Copy link
Owner

Choose a reason for hiding this comment

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

I think this tutorial notebook is broken in your branch, can you fix it or alternatively checkout the notebook from the main branch?

Copy link
Author

@mariamonzon mariamonzon Sep 26, 2025

Choose a reason for hiding this comment

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

You can select when merging but I will come back to the main branch

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.

2 participants