Skip to content

Commit ba86ea9

Browse files
committed
USD Converter: Added skipping when material does not exist
1 parent f791599 commit ba86ea9

File tree

3 files changed

+24
-1
lines changed

3 files changed

+24
-1
lines changed
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
<?xml verison="1.0"?>
2+
<robot name="simple_box_no_material">
3+
<link name="link1">
4+
<visual>
5+
<geometry>
6+
<box size="2.0 2.0 2.0"/>
7+
</geometry>
8+
</visual>
9+
</link>
10+
</robot>

tests/test_parser.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -222,6 +222,14 @@ def test_load_error_duplicate_transmission_names(self):
222222
with self.assertRaisesRegex(RuntimeError, r".*transmission: Transmission name 'transmission_1' already exists \(line: 8\).*"):
223223
parser.parse()
224224

225+
def test_has_no_material(self):
226+
# Load the specified URDF file.
227+
model_path = pathlib.Path("tests/data/simple_box_no_material.urdf")
228+
parser = URDFParser(model_path)
229+
230+
materials = parser.get_materials()
231+
self.assertEqual(len(materials), 0)
232+
225233
def test_get_basic_information(self):
226234
# Get basic information about a URDF.
227235

urdf_usd_converter/_impl/material.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@
88

99

1010
def convert_materials(data: ConversionData):
11-
# TODO: Implement
11+
materials = data.urdf_parser.get_materials()
12+
if not len(materials):
13+
return
14+
1215
data.libraries[Tokens.Materials] = usdex.core.addAssetLibrary(data.content[Tokens.Contents], Tokens.Materials, format="usdc")
1316
data.references[Tokens.Materials] = {}
17+
18+
# TODO: Implement

0 commit comments

Comments
 (0)