Repository navigation
Add docstring note that Transpose/Transposed do not update the affine matrix - #8888
Conversation
Signed-off-by: Akarsh Doki <doki.ak@northeastern.edu>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR adds documentation notes to the Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ericspod
left a comment
There was a problem hiding this comment.
Hi @Akarsh-Doki thanks for the added documentation, this is a good note to add for users.
Closes #6711.
Description
This PR adds a documentation note to the
TransposeandTransposedtransforms clarifying that they do not update the affine matrix in the
image metadata. As established in #5975, this is intended behavior:
applying an affine-dependent transform such as
Spacing/Spacingdafter
Transpose/Transposedcan therefore produce unexpected results.The note points users to
Orientation/Orientationdfor affine-awarereorientation.
This is a documentation-only change; no functional code is modified.
Types of changes
./runtests.sh -f -u --net --coverage../runtests.sh --quick --unittests --disttests.make htmlcommand in thedocs/folder.