robotology / idyntree

Multibody Dynamics Library designed for Free Floating Robots
BSD 3-Clause "New" or "Revised" License
155 stars 65 forks source link

Merge ***Raw classes in the non-Raw equivalents, and deprecate Raw classes #1114

Closed traversaro closed 8 months ago

traversaro commented 9 months ago

Some "logical" classes in iDynTree were split across two different classes, i.e. Rotation and RotationRaw , Position and PositionRaw , ... . This difference was there just for historical reason (related to semantic checks, that were removed in https://github.com/robotology/idyntree/pull/704), so by merging Raw classes with their non-Raw counterpart permit to simplify the codebase.

traversaro commented 9 months ago

There are a few CI failure, due to MATLAB tests failing. My plan is to merge https://github.com/robotology/idyntree/pull/1112 and then rebase this PR on top of that, and generating bindings, so we re-generated MATLAB bindings just once.

CLAassistant commented 8 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: traversaro
:x: github-actions[bot]
You have signed the CLA already but the status is still pending? Let us recheck it.

traversaro commented 8 months ago

The only failures are not related to the PR.