Skip to content

Commit ca30205

Browse files
committed
Fix test_batched_mounted_camera_rendering. The previous test case was wrong. It uses nenvs as index to be passed to attach. With the new implementation, index is not required. With batch rendering, cameras attached to links can be updated for each environment.
1 parent 04a5eee commit ca30205

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/test_render.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -279,8 +279,8 @@ def test_batched_mounted_camera_rendering(show_viewer, tol):
279279
T = np.eye(4)
280280
T[:3, :3] = np.array([[1, 0, 0], [0, -1, 0], [0, 0, -1]])
281281
T[:3, 3] = np.array([0.1, 0.0, 0.1])
282-
for nenv, cam in enumerate(cams):
283-
cam.attach(robot.get_link("hand"), T, nenv)
282+
for cam in cams:
283+
cam.attach(robot.get_link("hand"), T)
284284

285285
target_quat = np.tile(np.array([0, 1, 0, 0]), [n_envs, 1]) # pointing downwards
286286
center = np.tile(np.array([-0.25, -0.25, 0.5]), [n_envs, 1])

0 commit comments

Comments
 (0)