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

Displacement field bridge ITK-MONAI #8

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ntatsisk
Copy link

@ntatsisk ntatsisk commented Jan 30, 2023

Can this PR also become part of Project-MONAI/MONAI#5897, @Shadow-Devil @wyli?

cc. @thewtex @HastingsGreer

Related to: Project-MONAI/MONAI#4117

One important note: here I am using simple torch tensors as input for the wrapping functions. This is different from the affine bridge PR where MetaTensors where used. This means that there is a difference in the ordering of dimensions because of this line in the MetaTensor creation https://github.com/Project-MONAI/MONAI/blob/50c3f328cb98e4f0c4e6a38a04a98badf0961c18/monai/data/image_reader.py#L393. To be honest, I am not sure if Tensor or MetaTensor is preferable. Let me know what you think is better for the integration in the MONAI codebase.

--
xref:
Original affine bridge ITK-MONAI: #6
ITKElastix bridge milestone: InsightSoftwareConsortium/ITKElastix#126

@thewtex
Copy link
Member

thewtex commented Jan 30, 2023

@ntatsisk thanks!!

@HastingsGreer please take a look 🙏

Shadow-Devil added a commit to Shadow-Devil/MONAI that referenced this pull request Jan 30, 2023
…and automatically download test data

Signed-off-by: Felix Schnabel <[email protected]>
@Shadow-Devil
Copy link

I tried to add it into the PR. If you have any feedback, I would highly appreciate it 😄

Shadow-Devil added a commit to Shadow-Devil/MONAI that referenced this pull request Jan 30, 2023
…and automatically download test data

Signed-off-by: Felix Schnabel <[email protected]>
Shadow-Devil added a commit to Shadow-Devil/MONAI that referenced this pull request Feb 4, 2023
…and automatically download test data

Signed-off-by: Felix Schnabel <[email protected]>
return np.asarray(warped_image)


def monai_wrap(image_tensor, ddf_tensor):

Choose a reason for hiding this comment

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

typo wrap -> warp ?

Shadow-Devil added a commit to Shadow-Devil/MONAI that referenced this pull request Feb 7, 2023
…and automatically download test data

Signed-off-by: Felix Schnabel <[email protected]>
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.

5 participants