Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add InitialTransformParameterObject maps to registration result #898

Conversation

N-Dekker
Copy link
Member

Added the maps from the InitialTransformParameterObject of an ElastixRegistrationMethod to the output TransformParameterObject (at the begin, before the maps added during registration).

Ensures that the initial transform is included, when the result from registration.GetTransformParameterObject() is passed to transformix.SetTransformParameterObject.

@N-Dekker N-Dekker changed the title ENH: InitialTransformParameterObject maps to TransformParameterObject Add InitialTransformParameterObject maps to registration result May 24, 2023
@N-Dekker N-Dekker force-pushed the InitialTransformParameterObject-maps-to-TransformParameterObject branch 3 times, most recently from d029bc9 to c48d73d Compare May 24, 2023 19:04
Added the maps from the InitialTransformParameterObject of an ElastixRegistrationMethod to the output TransformParameterObject (at the begin, before the maps added during registration).

Ensures that the initial transform is included, when the result from `registration.GetTransformParameterObject()` is passed to `transformix.SetTransformParameterObject`. Discussed with Marius (mstaring).
@N-Dekker N-Dekker force-pushed the InitialTransformParameterObject-maps-to-TransformParameterObject branch from c48d73d to 50a5696 Compare May 25, 2023 11:19
@N-Dekker N-Dekker merged commit 691c358 into main May 25, 2023
@N-Dekker N-Dekker deleted the InitialTransformParameterObject-maps-to-TransformParameterObject branch May 25, 2023 13:12
N-Dekker added a commit to N-Dekker/ITKElastix that referenced this pull request Jul 19, 2023
Follow-up to InsightSoftwareConsortium#211
commit 1043f10 "ENH: Bump elastix version to
2023-05-15, use C++17"

Including:

  SuperElastix/elastix#936
  SuperElastix/elastix@115d8e1
  ENH: Add `ElastixRegistrationMethod::SetInitialTransform

  SuperElastix/elastix#933
  SuperElastix/elastix@d33a0d5
  BUG: Fix reference count in ConvertItkTransformBaseToSingleItkTransform

  SuperElastix/elastix#924
  SuperElastix/elastix@71a4a9a
  ENH: Add TransformParameterFileName property to TransformixFilter

  SuperElastix/elastix#922
  SuperElastix/elastix@9d76b86
  ENH: Restore ELASTIX_BUILD_EXECUTABLE CMake option

  SuperElastix/elastix#909
  SuperElastix/elastix@9e64269
  ENH: ElastixRegistrationMethod writes initial transform parameter files

  SuperElastix/elastix#903
  SuperElastix/elastix@2170592
  ENH: Remove the letter 's' from "InitialTransformParametersFileName"

  SuperElastix/elastix#898
  SuperElastix/elastix@691c358
  BUG: Add InitialTransformParameterObject maps to registration result
N-Dekker added a commit to N-Dekker/ITKElastix that referenced this pull request Jul 20, 2023
Follow-up to InsightSoftwareConsortium#211
commit 1043f10 "ENH: Bump elastix version to
2023-05-15, use C++17"

Including:

  SuperElastix/elastix#936
  SuperElastix/elastix@115d8e1
  ENH: Add `ElastixRegistrationMethod::SetInitialTransform

  SuperElastix/elastix#933
  SuperElastix/elastix@d33a0d5
  BUG: Fix reference count in ConvertItkTransformBaseToSingleItkTransform

  SuperElastix/elastix#924
  SuperElastix/elastix@71a4a9a
  ENH: Add TransformParameterFileName property to TransformixFilter

  SuperElastix/elastix#922
  SuperElastix/elastix@9d76b86
  ENH: Restore ELASTIX_BUILD_EXECUTABLE CMake option

  SuperElastix/elastix#909
  SuperElastix/elastix@9e64269
  ENH: ElastixRegistrationMethod writes initial transform parameter files

  SuperElastix/elastix#903
  SuperElastix/elastix@2170592
  ENH: Remove the letter 's' from "InitialTransformParametersFileName"

  SuperElastix/elastix#898
  SuperElastix/elastix@691c358
  BUG: Add InitialTransformParameterObject maps to registration result
N-Dekker added a commit to N-Dekker/ITKElastix that referenced this pull request Jul 25, 2023
Follow-up to InsightSoftwareConsortium#211
commit 1043f10 "ENH: Bump elastix version to
2023-05-15, use C++17"

Including:

  SuperElastix/elastix#936
  SuperElastix/elastix@115d8e1
  ENH: Add `ElastixRegistrationMethod::SetInitialTransform

  SuperElastix/elastix#933
  SuperElastix/elastix@d33a0d5
  BUG: Fix reference count in ConvertItkTransformBaseToSingleItkTransform

  SuperElastix/elastix#924
  SuperElastix/elastix@71a4a9a
  ENH: Add TransformParameterFileName property to TransformixFilter

  SuperElastix/elastix#922
  SuperElastix/elastix@9d76b86
  ENH: Restore ELASTIX_BUILD_EXECUTABLE CMake option

  SuperElastix/elastix#909
  SuperElastix/elastix@9e64269
  ENH: ElastixRegistrationMethod writes initial transform parameter files

  SuperElastix/elastix#903
  SuperElastix/elastix@2170592
  ENH: Remove the letter 's' from "InitialTransformParametersFileName"

  SuperElastix/elastix#898
  SuperElastix/elastix@691c358
  BUG: Add InitialTransformParameterObject maps to registration result
N-Dekker added a commit to InsightSoftwareConsortium/ITKElastix that referenced this pull request Jul 25, 2023
Follow-up to #211
commit 1043f10 "ENH: Bump elastix version to
2023-05-15, use C++17"

Including:

  SuperElastix/elastix#936
  SuperElastix/elastix@115d8e1
  ENH: Add `ElastixRegistrationMethod::SetInitialTransform

  SuperElastix/elastix#933
  SuperElastix/elastix@d33a0d5
  BUG: Fix reference count in ConvertItkTransformBaseToSingleItkTransform

  SuperElastix/elastix#924
  SuperElastix/elastix@71a4a9a
  ENH: Add TransformParameterFileName property to TransformixFilter

  SuperElastix/elastix#922
  SuperElastix/elastix@9d76b86
  ENH: Restore ELASTIX_BUILD_EXECUTABLE CMake option

  SuperElastix/elastix#909
  SuperElastix/elastix@9e64269
  ENH: ElastixRegistrationMethod writes initial transform parameter files

  SuperElastix/elastix#903
  SuperElastix/elastix@2170592
  ENH: Remove the letter 's' from "InitialTransformParametersFileName"

  SuperElastix/elastix#898
  SuperElastix/elastix@691c358
  BUG: Add InitialTransformParameterObject maps to registration result
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.

1 participant