Skip to content

Commit

Permalink
fixed save/load test
Browse files Browse the repository at this point in the history
  • Loading branch information
rgerum committed Sep 21, 2021
1 parent 130ac6f commit 330e7f2
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 6 deletions.
2 changes: 1 addition & 1 deletion cameratransform/camera.py
Original file line number Diff line number Diff line change
Expand Up @@ -1434,7 +1434,7 @@ def save(self, filename):
the filename where to store the parameters.
"""
keys = self.parameters.parameters.keys()
export_dict = {key: getattr(self, key) for key in keys}
export_dict = {key: getattr(self, key) for key in keys if key != "focallength_px"}

# check projections and save
if isinstance(self.projection, RectilinearProjection):
Expand Down
2 changes: 1 addition & 1 deletion cameratransform/projection.py
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ def __str__(self):

def save(self, filename):
keys = self.parameters.parameters.keys()
export_dict = {key: getattr(self, key) for key in keys}
export_dict = {key: getattr(self, key) for key in keys if key != "focallength_px"}
with open(filename, "w") as fp:
fp.write(json.dumps(export_dict))

Expand Down
11 changes: 7 additions & 4 deletions tests/test_transforms.py
Original file line number Diff line number Diff line change
Expand Up @@ -130,13 +130,15 @@ def test_saveLoad(self, cam):
cam.save(filename)
cam2 = ct.load_camera(filename)
for key in cam.parameters.parameters:
self.assertAlmostEqual(getattr(cam, key), getattr(cam2, key), 3)
if key != "focallength_px":
self.assertAlmostEqual(getattr(cam, key), getattr(cam2, key), 3)

cam.projection.save(filename)
cam2.focallength = 999
cam2.focallength_px = 999
cam2.projection.load(filename)
for key in cam.parameters.parameters:
self.assertAlmostEqual(getattr(cam, key), getattr(cam2, key), 3)
if key != "focallength_px":
self.assertAlmostEqual(getattr(cam, key), getattr(cam2, key), 3)

cam.orientation.save(filename)
cam2.elevation_m = 9
Expand All @@ -145,7 +147,8 @@ def test_saveLoad(self, cam):
cam2.roll_deg = 9
cam2.orientation.load(filename)
for key in cam.orientation.parameters.parameters:
self.assertAlmostEqual(getattr(cam, key), getattr(cam2, key), 3)
if key != "focallength_px":
self.assertAlmostEqual(getattr(cam, key), getattr(cam2, key), 3)

@given(ct_st.camera())
def test_print(self, cam):
Expand Down

0 comments on commit 330e7f2

Please sign in to comment.