Skip to content

Commit 51f6aa8

Browse files
quaglacopybara-github
authored andcommitted
Store assets by address instead of value in MjSpec.
PiperOrigin-RevId: 733343350 Change-Id: I42ee2a184ebff37a080dace0422bb61e1ba53527
1 parent 7d9df8c commit 51f6aa8

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

python/mujoco/specs_test.py

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
# ==============================================================================
1515
"""Tests for mjSpec bindings."""
1616

17+
import gc
1718
import inspect
1819
import os
1920
import textwrap
@@ -750,6 +751,10 @@ def test_assets(self):
750751
model = spec.compile()
751752
self.assertEqual(model.nmeshvert, 8)
752753
self.assertEqual(spec.assets['cube.obj'], cube)
754+
self.assertIs(
755+
spec.assets['cube.obj'], cube,
756+
'Asset dict should contain a reference, not a copy'
757+
)
753758

754759
xml = """
755760
<mujoco model="test">
@@ -766,6 +771,13 @@ def test_assets(self):
766771
model = spec.compile()
767772
self.assertEqual(model.nmeshvert, 8)
768773
self.assertEqual(spec.assets['cube.obj'], cube)
774+
self.assertIs(
775+
spec.assets['cube.obj'], cube,
776+
'Asset dict should contain a reference, not a copy'
777+
)
778+
del assets
779+
gc.collect()
780+
self.assertEqual(spec.assets['cube.obj'], cube)
769781

770782
def test_include(self):
771783
included_xml = """

0 commit comments

Comments
 (0)