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

PERF: Fill jsh (JacobianOfSpatialJacobian) in-place and remove jsh1 #890

Merged
merged 1 commit into from
May 10, 2023

Conversation

N-Dekker
Copy link
Member

Used modern C++ range-based for loops to iterate over the data of jsh. Reduced dynamic memory usage by removing jsh1.

Used modern C++ range-based `for` loops to iterate over the data of `jsh`. Reduced dynamic memory usage by removing `jsh1`.
@N-Dekker N-Dekker merged commit b9ea3a8 into main May 10, 2023
@N-Dekker N-Dekker deleted the Fill-jsh-in-place branch May 10, 2023 18:12
N-Dekker added a commit that referenced this pull request May 11, 2023
Used modern C++ range-based `for` loops to iterate over the matrices of `jsj`, in both `GetJacobianOfSpatialJacobianUseComposition` overloads. Reduced dynamic memory usage by removing `jsj1` in those two cases.

Follow-up to pull request #890 commit b9ea3a8
N-Dekker added a commit that referenced this pull request May 11, 2023
Used modern C++ range-based `for` loops to iterate over the matrices of `jsj`, in both `GetJacobianOfSpatialJacobianUseComposition` overloads. Reduced dynamic memory usage by removing `jsj1` in those two cases.

Follow-up to pull request #890 commit b9ea3a8
N-Dekker added a commit that referenced this pull request May 11, 2023
Used modern C++ range-based `for` loops to iterate over the matrices of `jsj`, in both `GetJacobianOfSpatialJacobianUseComposition` overloads. Reduced dynamic memory usage by removing `jsj1` in those two cases.

Follow-up to pull request #890 commit b9ea3a8
N-Dekker added a commit to N-Dekker/ITKElastix that referenced this pull request May 16, 2023
Including:

  pull request SuperElastix/elastix#891
  commit SuperElastix/elastix@ed15547
  "PERF: Fill `jsj` (JacobianOfSpatialJacobian) in-place and remove `jsj1`"

  pull request SuperElastix/elastix#890
  commit SuperElastix/elastix@b9ea3a8
  "PERF: Fill `jsh` (JacobianOfSpatialJacobian) in-place and remove `jsh1`"

  pull request SuperElastix/elastix#887
  commit SuperElastix/elastix@8298485
  PERF: Make EvaluateParzenValues calls faster, using raw buffer of values

  pull request SuperElastix/elastix#882
  commit SuperElastix/elastix@58e0a7b
  "ENH: Convert the input images to the user-specified internal pixel type"

  pull request SuperElastix/elastix#864
  commit SuperElastix/elastix@c3d478e
  "ENH: Upgrade elastix from C++14 to C++17"

  pull request SuperElastix/elastix#856
  commit SuperElastix/elastix@48c6458
  "ENH: Add SetInitialTransformParameterObject to ElastixRegistrationMethod"

  pull request SuperElastix/elastix#832
  commit SuperElastix/elastix@05d2b40
  ENH: Support "ShowProgressPercentage" parameter (`false` by default)

  pull request SuperElastix/elastix#815
  commit SuperElastix/elastix@c4ef707
  "ENH: Add `ElastixLogLevel` to the ITK interface"

Explicitly specified C++17 as standard for the compilation of ITKElastix.
N-Dekker added a commit to N-Dekker/ITKElastix that referenced this pull request May 16, 2023
Including:

  pull request SuperElastix/elastix#891
  commit SuperElastix/elastix@ed15547
  "PERF: Fill `jsj` (JacobianOfSpatialJacobian) in-place and remove `jsj1`"

  pull request SuperElastix/elastix#890
  commit SuperElastix/elastix@b9ea3a8
  "PERF: Fill `jsh` (JacobianOfSpatialJacobian) in-place and remove `jsh1`"

  pull request SuperElastix/elastix#887
  commit SuperElastix/elastix@8298485
  PERF: Make EvaluateParzenValues calls faster, using raw buffer of values

  pull request SuperElastix/elastix#882
  commit SuperElastix/elastix@58e0a7b
  "ENH: Convert the input images to the user-specified internal pixel type"

  pull request SuperElastix/elastix#864
  commit SuperElastix/elastix@c3d478e
  "ENH: Upgrade elastix from C++14 to C++17"

  pull request SuperElastix/elastix#856
  commit SuperElastix/elastix@48c6458
  "ENH: Add SetInitialTransformParameterObject to ElastixRegistrationMethod"

  pull request SuperElastix/elastix#832
  commit SuperElastix/elastix@05d2b40
  ENH: Support "ShowProgressPercentage" parameter (`false` by default)

  pull request SuperElastix/elastix#815
  commit SuperElastix/elastix@c4ef707
  "ENH: Add `ElastixLogLevel` to the ITK interface"

Explicitly specified C++17 as standard for the compilation of ITKElastix.
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