Feature/manifoldtwojacobians
Frank Dellaert
Branch: feature/ManifoldTwoJacobians
Branch: develop
Declined
Declined pull request
This change would make definitions in TRO paper different from the implementation.
Closed by: Frank Dellaert·2018-12-30
This change would make definitions in TRO paper different from the implementation.
Luca, it would be great if we could push this through. Maybe spend some thinking about the meaning of the Jacobians and how we can fix up this code to pass the tests? I merged in develop, got rid off old Jacobians, and everything compiles... Stuff still needs to move to the base class if you can get it to work.