Skip to content

Commit

Permalink
Insert back comment on urdf_parser_py fix
Browse files Browse the repository at this point in the history
  • Loading branch information
Giulero committed Jun 12, 2024
1 parent 537db7c commit 0e21593
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions src/adam/model/std_factories/std_model.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,13 @@ def __init__(self, path: str, math: SpatialMath):
if not path.exists():
raise FileExistsError(path)

# Read URDF, but before passing it to urdf_parser_py get rid of all sensor tags
# sensor tags are valid elements of URDF (see ),
# but they are ignored by urdf_parser_py, that complains every time it sees one.
# As there is nothing to be fixed in the used models, and it is not useful
# to have a useless and noisy warning, let's remove before hands all the sensor elements,
# that anyhow are not parser by urdf_parser_py or adam
# See https://github.com/ami-iit/ADAM/issues/59
with open(path, "r") as xml_file:
xml_string = xml_file.read()
xml_string_without_sensors_tags = urdf_remove_sensors_tags(xml_string)
Expand Down

0 comments on commit 0e21593

Please sign in to comment.