gazebosim / gz-mujoco

25 stars 4 forks source link

Load MJCF files with `escape_separators=True` #72

Closed azeey closed 2 years ago

azeey commented 2 years ago

🦟 Bug fix

Summary

This ensures that roundtrip tests where we convert an SDFormat file to MJCF and then back to SDFormat don't fail when reading the MJCF file.

Checklist

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

codecov[bot] commented 2 years ago

Codecov Report

Merging #72 (9015860) into main (212283e) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   97.22%   97.22%           
=======================================
  Files          19       19           
  Lines         828      828           
=======================================
  Hits          805      805           
  Misses         23       23           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 212283e...9015860. Read the comment docs.