Skip to content

Improve documentation for composite transform. - #2628

Merged
zivy merged 1 commit into
SimpleITK:mainfrom
zivy:updateDocs
Jul 14, 2026
Merged

zivy merged 1 commit into
SimpleITK:mainfrom
zivy:updateDocs

Conversation

@zivy

@zivy zivy commented Jul 14, 2026

Copy link
Copy Markdown
Member

Improve the documentation for the composite transform and add a note about how the order of transforms
relates to the order used in the ANTs ecosystem.

@blowekamp blowekamp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks concise and clear to me.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the SimpleITK fundamentals documentation to clarify CompositeTransform ordering semantics and to help users translate transform ordering expectations when moving between SimpleITK and the ANTs toolchain.

Changes:

  • Expanded the composite transform documentation with clearer “first added, last applied” examples.
  • Added an ANTs-focused note describing the transform ordering relationship with antsApplyTransforms.
  • Extended the workflow spellcheck dictionary with Tn.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
docs/source/fundamentalConcepts.rst Clarifies composite transform stacking semantics and adds an ANTs ordering note with examples.
.github/workflows/additional_dictionary.txt Adds Tn to the additional dictionary to avoid false positives in spellchecking.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/source/fundamentalConcepts.rst Outdated
Comment thread docs/source/fundamentalConcepts.rst Outdated
Comment thread docs/source/fundamentalConcepts.rst Outdated
Improve the documentation for the composite transform
and add a note about how the order of transforms
relates to the order used in the ANTs ecosystem.

Co-authored-by: Soumya Snigdha Kundu <[email protected]>
@gdevenyi

Copy link
Copy Markdown
Contributor

I'm happy with this.

@zivy
zivy merged commit b6579e5 into SimpleITK:main Jul 14, 2026
3 checks passed
@zivy
zivy deleted the updateDocs branch July 14, 2026 18:53
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.

4 participants