Expose Sophus::interpolate to Python bindings for SO2/3 and SE2/3#560
Open
SidArora29 wants to merge 2 commits into
Open
Expose Sophus::interpolate to Python bindings for SO2/3 and SE2/3#560SidArora29 wants to merge 2 commits into
SidArora29 wants to merge 2 commits into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR exposes the
Sophus::interpolatefunction to the Python bindings for the following Lie Groups:SO2andSO3SE2andSE3Interpolation on the manifold is a key utility for state estimation and perception workflows in Python, such as:
Implementation Details
The template function
Sophus::interpolateis bound at the module level insophus_pybind/bindings.cppfor the double-precision variants, maintaining consistency with existing bound types.Testing
Verified the implementation locally by confirming that the logarithmic map of the interpolated midpoint matches the expected geodesic midpoint on the manifold: