Skip to content

Commit 3e6352c

Browse files
PranjalSahuhjmjohnson
authored andcommitted
BUG: First set fixed parameters then parameters
First fixed parameters should be set and then parameters as based on fixed parameters additional calculations are done.
1 parent 963d0a8 commit 3e6352c

File tree

1 file changed

+9
-8
lines changed
  • Wrapping/Generators/Python/itk/support

1 file changed

+9
-8
lines changed

Wrapping/Generators/Python/itk/support/extras.py

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1083,17 +1083,18 @@ def transform_from_dict(transform_dict: Dict)-> "itkt.TransformBase":
10831083
import itk
10841084

10851085
def set_parameters(transform, transform_parameters, transform_fixed_parameters):
1086-
o1 = transform.GetParameters()
1087-
o1.SetSize(transform_parameters.shape[0])
1088-
for j, v in enumerate(transform_parameters):
1086+
# First set fixed parameters then parameters
1087+
o1 = transform.GetFixedParameters()
1088+
o1.SetSize(transform_fixed_parameters.shape[0])
1089+
for j, v in enumerate(transform_fixed_parameters):
10891090
o1.SetElement(j, v)
1090-
transform.SetParameters(o1)
1091+
transform.SetFixedParameters(o1)
10911092

1092-
o2 = transform.GetFixedParameters()
1093-
o2.SetSize(transform_fixed_parameters.shape[0])
1094-
for j, v in enumerate(transform_fixed_parameters):
1093+
o2 = transform.GetParameters()
1094+
o2.SetSize(transform_parameters.shape[0])
1095+
for j, v in enumerate(transform_parameters):
10951096
o2.SetElement(j, v)
1096-
transform.SetFixedParameters(o2)
1097+
transform.SetParameters(o2)
10971098

10981099

10991100
# For checking transforms which don't take additional parameters while instantiation

0 commit comments

Comments
 (0)