Skip to content

4281 - adds zero_centered option to AffineTransform#4282

Merged
wyli merged 6 commits intoProject-MONAI:devfrom
wyli:4281-norm-mode
May 18, 2022
Merged

4281 - adds zero_centered option to AffineTransform#4282
wyli merged 6 commits intoProject-MONAI:devfrom
wyli:4281-norm-mode

Conversation

@wyli
Copy link
Copy Markdown
Contributor

@wyli wyli commented May 17, 2022

Signed-off-by: Wenqi Li [email protected]

Fixes #4281

  • currently AffineTransform only allows for centre of rotation from the image origin.
    this PR adds the support of rotation around the spatial centre of the input image.
  • update Affine's norm_coords option to be consistent with AffineTransforms

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

wyli added 5 commits May 17, 2022 10:01
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli marked this pull request as ready for review May 17, 2022 16:25
@wyli wyli requested review from Nic-Ma, ericspod, rijobro and yiheng-wang-nv and removed request for Nic-Ma May 17, 2022 16:25
Copy link
Copy Markdown
Contributor

@Nic-Ma Nic-Ma left a comment

Choose a reason for hiding this comment

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

Thanks for the quick update.
Looks good to me, put some minor comments inline.

Thanks.

Signed-off-by: Wenqi Li <[email protected]>
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented May 18, 2022

/build

@wyli wyli enabled auto-merge (squash) May 18, 2022 12:18
@wyli wyli merged commit cd77e91 into Project-MONAI:dev May 18, 2022
@wyli wyli deleted the 4281-norm-mode branch June 11, 2022 09:36
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.

Inconsistency networks.layer.AffineTransform and transforms.Affine

2 participants