search

tesseract-robotics / tesseract

Motion Planning Environment
http://tesseract-docs.rtfd.io
Other
278 stars 88 forks source link

Handle edge case in calcJacobianTransformErrorDiff #1022

Closed Levi-Armstrong closed 4 months ago

Levi-Armstrong commented 4 months ago

Address #1020

Levi-Armstrong commented 4 months ago

@marrts Can you test and confirm this solves the issue you were having?

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.84%. Comparing base (0de0151) to head (e44b7c3).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/tesseract-robotics/tesseract/pull/1022/graphs/tree.svg?width=650&height=150&src=pr&token=nh4aHZzgpR&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tesseract-robotics)](https://app.codecov.io/gh/tesseract-robotics/tesseract/pull/1022?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tesseract-robotics) ```diff @@ Coverage Diff @@ ## master #1022 +/- ## ======================================= Coverage 89.84% 89.84% ======================================= Files 280 280 Lines 15908 15909 +1 ======================================= + Hits 14293 14294 +1 Misses 1615 1615 ``` [see 2 files with indirect coverage changes](https://app.codecov.io/gh/tesseract-robotics/tesseract/pull/1022/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tesseract-robotics)