Skip to content

Commit a65e9ce

Browse files
committed
rename MixedDim to MixedDimKinematic
1 parent c76c7b8 commit a65e9ce

File tree

3 files changed

+7
-8
lines changed

3 files changed

+7
-8
lines changed

src/torchlensmaker/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161

6262
from torchlensmaker.kinematics.kinematics_elements import (
6363
KinematicElement,
64-
MixedDim,
64+
MixedDimKinematic,
6565
AbsolutePosition,
6666
Gap,
6767
Rotate3D,

src/torchlensmaker/kinematics/kinematics_elements.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -176,8 +176,7 @@ def forward(
176176
return self.func.forward(dfk, ifk, self.y, self.z)
177177

178178

179-
class MixedDim(KinematicElement):
180-
# TODO true mixed dim that works with any element?
179+
class MixedDimKinematic(KinematicElement):
181180
def __init__(self, module_2d: nn.Module, module_3d: nn.Module):
182181
super().__init__()
183182
self.module_2d = module_2d
@@ -195,7 +194,7 @@ def __init__(self, offset: Float[torch.Tensor, ""] | float | int):
195194
super().__init__()
196195
translate_2d = Translate2D(x=offset)
197196
translate_3d = Translate3D(x=offset)
198-
self.mixed_dim = MixedDim(translate_2d, translate_3d)
197+
self.mixed_dim = MixedDimKinematic(translate_2d, translate_3d)
199198

200199
def forward(self, dfk: HomMatrix, ifk: HomMatrix) -> tuple[HomMatrix, HomMatrix]:
201200
return self.mixed_dim(dfk, ifk)
@@ -206,7 +205,7 @@ def __init__(
206205
self, angles: tuple[float | int, float | int] | Float[torch.Tensor, "2"]
207206
):
208207
super().__init__()
209-
self.mixed_dim = MixedDim(Rotate2D(angles[0]), Rotate3D(angles[1], angles[0]))
208+
self.mixed_dim = MixedDimKinematic(Rotate2D(angles[0]), Rotate3D(angles[1], angles[0]))
210209

211210
def forward(self, dfk: HomMatrix, ifk: HomMatrix) -> tuple[HomMatrix, HomMatrix]:
212211
return self.mixed_dim(dfk, ifk)
@@ -220,7 +219,7 @@ def __init__(
220219
z: Float[torch.Tensor, ""] | float | int = 0.0,
221220
):
222221
super().__init__()
223-
self.mixed_dim = MixedDim(Translate2D(x, y), Translate3D(x, y, z))
222+
self.mixed_dim = MixedDimKinematic(Translate2D(x, y), Translate3D(x, y, z))
224223

225224
def forward(self, dfk: HomMatrix, ifk: HomMatrix) -> tuple[HomMatrix, HomMatrix]:
226225
return self.mixed_dim(dfk, ifk)

src/torchlensmaker/kinematics/tests/test_kinematics_elements.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
Rotate3D,
3939
AbsolutePosition,
4040
AbsolutePositionVec3D,
41-
MixedDim,
41+
MixedDimKinematic,
4242
Gap,
4343
Rotate,
4444
Translate,
@@ -243,7 +243,7 @@ def test_elements_mixed() -> None:
243243

244244
chain_model_mixed = nn.ModuleList(
245245
[
246-
MixedDim(
246+
MixedDimKinematic(
247247
Rotate2D(0.1),
248248
Rotate3D(y=0.1),
249249
),

0 commit comments

Comments
 (0)