Skip to content

Add key word only options to ReadImage - #2296

Merged
zivy merged 1 commit into
SimpleITK:masterfrom
blowekamp:add_kwargs_reader
May 29, 2025
Merged

zivy merged 1 commit into
SimpleITK:masterfrom
blowekamp:add_kwargs_reader

Conversation

@blowekamp

@blowekamp blowekamp commented May 16, 2025 •

Copy link
Copy Markdown
Member

Enables advance options like ReverseOrder, and Debug to be set with the procedural ReadImage interface

@blowekamp
blowekamp force-pushed the add_kwargs_reader branch from 88911b8 to eb1cacb Compare May 22, 2025 15:31
Enables advance options like ReverseOrder, and Debug to be set with
the procedural interface.
@blowekamp
blowekamp force-pushed the add_kwargs_reader branch from eb1cacb to 2a944a6 Compare May 22, 2025 18:06
@blowekamp blowekamp changed the title WIP: Add key word only options to ReadImage Add key word only options to ReadImage May 22, 2025
@blowekamp
blowekamp marked this pull request as ready for review May 22, 2025 19:46
@blowekamp
blowekamp requested review from dave3d and zivy May 22, 2025 19:46

@zivy zivy 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.

Elegant implementation with the keywords configured to invoke "Set" methods using introspection.

@zivy
zivy merged commit a10127c into SimpleITK:master May 29, 2025
@blowekamp
blowekamp deleted the add_kwargs_reader branch September 9, 2026 12:55
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.

3 participants