Fixed index out of bounds at body_jacobians #764
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.
My program panicked at here, then I added an
assert
like this:And it just failed. I didn't unstand these code well, but I noticed the
internal_id
of the first new link will be updated toself.links.len()
, then I gaussedbase_internal_id
should be set toself.links.len()
, or there will be an unusedinternal_id
. I tested this change with my program and theMultibodyJoint
example, it seemed not bad.