Skip to content

ENH: Rename ResampleInPlaceImage to TransformGeometryImageFilter#2723

Merged
blowekamp merged 1 commit intoInsightSoftwareConsortium:masterfrom
blowekamp:TransformGeometryImageFilter
Sep 9, 2021
Merged

ENH: Rename ResampleInPlaceImage to TransformGeometryImageFilter#2723
blowekamp merged 1 commit intoInsightSoftwareConsortium:masterfrom
blowekamp:TransformGeometryImageFilter

Conversation

@blowekamp
Copy link
Member

Community discussion and voting converged TransformGeometryImageFilter
is a more accurate name for the operation.

Renamed per the discussion: https://discourse.itk.org/t/simpleitk-euler3d-transform-adjust-image-origin-and-direction/3966/25

@github-actions github-actions bot added area:Core Issues affecting the Core module area:Python wrapping Python bindings for a class type:Enhancement Improvement of existing methods or implementation type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct labels Sep 8, 2021
Copy link
Member

@hjmjohnson hjmjohnson left a comment

Choose a reason for hiding this comment

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

This is great! You also had identified bugs in pipeline processing. Will you be addressing those issues as well?

@blowekamp
Copy link
Member Author

@hjmjohnson Yes, I am working on those now. I am prioritizing API changes to get them merged before the RC. Internal corrections and pipeline fixes are coming.

@hjmjohnson
Copy link
Member

@hjmjohnson Yes, I am working on those now. I am prioritizing API changes to get them merged before the RC. Internal corrections and pipeline fixes are coming.

You are awesome! Thank you.

@blowekamp blowekamp force-pushed the TransformGeometryImageFilter branch from 7863712 to f87bff1 Compare September 8, 2021 17:46
@dzenanz
Copy link
Member

dzenanz commented Sep 8, 2021

I guess merge as soon as the CI comes back green?

Community discussion and voting converged TransformGeometryImageFilter
is a more accurate name for the operation.
@blowekamp blowekamp force-pushed the TransformGeometryImageFilter branch from f87bff1 to 9827449 Compare September 8, 2021 20:24
@blowekamp blowekamp merged commit c2b94a4 into InsightSoftwareConsortium:master Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Core Issues affecting the Core module area:Python wrapping Python bindings for a class type:Enhancement Improvement of existing methods or implementation type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants