Fix/include pose degrees

[aaronchongth]

🦟 Bug fix

Fixes #696

Summary

Pose in include tags are parsed in a single readXml call, instead of recursively. Added parsing logic for attribute degrees alongside relative_to.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

[scpeters]

nit: was it necessary to reorder the statements in parser.cc? I think the code is just moved around, but it makes the diff a bit harder to read

[aaronchongth]

@scpeters, yes the reorder was on purpose. Values are now parsed dependent on the attribute @degrees, if I kept the order, I would need to call Reparse() on the underlying value Param for the pose Element, after the attribute has been parsed.

[scpeters]

@scpeters, yes the reorder was on purpose. Values are now parsed dependent on the attribute @degrees, if I kept the order, I would need to call Reparse() on the underlying value Param for the pose Element, after the attribute has been parsed.

thanks for clarifying; I hadn't reviewed the changes in #589, but I just looked and saw how it changed Param::SetFromString so this makes more sense now