Skip to content

BUG: Add missing single argument ImageAdaptor Transform member functions#2889

Merged
dzenanz merged 1 commit intoInsightSoftwareConsortium:masterfrom
N-Dekker:ImageAdaptor-single-parameter-conversion-functions
Nov 23, 2021
Merged

BUG: Add missing single argument ImageAdaptor Transform member functions#2889
dzenanz merged 1 commit intoInsightSoftwareConsortium:masterfrom
N-Dekker:ImageAdaptor-single-parameter-conversion-functions

Conversation

@N-Dekker
Copy link
Copy Markdown
Contributor

@N-Dekker N-Dekker commented Nov 20, 2021

Added the four missing single parameter overloads of point/index/vector
conversion member functions to ImageAdaptor, which were already there
for ImageBase with ITK version 5.0:

TransformPhysicalPointToIndex
TransformPhysicalPointToContinuousIndex
TransformContinuousIndexToPhysicalPoint
TransformIndexToPhysicalPoint

Included a GoogleTest unit test.

Follow-up to pull request #868
commit 65233e3
"ENH: Convenience overloads for ImageBase Transform member functions" (by me)

And pull request #1207
commit 55f8ac2
"BUG: Missing ImageAdapter function signatures" by Hans Johnson (@hjmjohnson)

@github-actions github-actions Bot added area:Core Issues affecting the Core module type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances 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 Nov 20, 2021
@N-Dekker N-Dekker force-pushed the ImageAdaptor-single-parameter-conversion-functions branch 5 times, most recently from a798cf8 to 810d2ea Compare November 21, 2021 10:12
Added the four missing single parameter overloads of point/index/vector
conversion member functions to `ImageAdaptor`, which were already there
for `ImageBase` with ITK version 5.0:

    TransformPhysicalPointToIndex
    TransformPhysicalPointToContinuousIndex
    TransformContinuousIndexToPhysicalPoint
    TransformIndexToPhysicalPoint

Included a GoogleTest unit test.

Follow-up to pull request InsightSoftwareConsortium#868
commit 65233e3
"ENH: Convenience overloads for ImageBase Transform member functions"

And pull request InsightSoftwareConsortium#1207
commit 55f8ac2
"BUG: Missing ImageAdapter function signatures"
@N-Dekker N-Dekker force-pushed the ImageAdaptor-single-parameter-conversion-functions branch from 810d2ea to 5fa53b4 Compare November 21, 2021 10:16
@N-Dekker N-Dekker marked this pull request as ready for review November 21, 2021 13:03
@N-Dekker N-Dekker requested a review from hjmjohnson November 23, 2021 17:24
@dzenanz dzenanz merged commit 2d06a6a into InsightSoftwareConsortium:master Nov 23, 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 type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances 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.

2 participants