Update changelog

CHANGELOG.md

@@ -13,6 +13,16 @@ Release Versions:
 - Add CI workflow for Python bindings (#159)
 - Add emtpy constructors for Circular and Ring DS (#154)
 - Update ROS1 example because simulation is more developed (#160)
+- Define `inverse` and `*` operators for Cartesian types explicitly (#158)
+
+## Important TODOs
+
+- Revise `*=` and `*` operators in Cartesian types before the next release with 
+  breaking changes (some are marked *deprecated*, and some are left as is, but 
+  they should be deleted). See issue #156
+- Add the wrench computation in the `*` operator and `inverse` function (#134)
+- Refactor and improve unittests for state_representation (especially JointState
+  and CartesianState)
 
 ## 3.0.0
 

source/state_representation/src/space/cartesian/CartesianTwist.cpp

@@ -164,8 +164,7 @@ Eigen::VectorXd CartesianTwist::data() const {
 }
 
 CartesianTwist CartesianTwist::inverse() const {
-  CartesianTwist result = this->CartesianState::inverse();
-  return result;
+  return this->CartesianState::inverse();
 }
 
 std::ostream& operator<<(std::ostream& os, const CartesianTwist& twist) {
@@ -184,7 +183,7 @@ std::ostream& operator<<(std::ostream& os, const CartesianTwist& twist) {
 }
 
 CartesianTwist operator*(const CartesianState& state, const CartesianTwist& twist) {
-  return CartesianTwist(state.operator*(twist));
+  return state.operator*(twist);
 }
 
 CartesianTwist operator*(double lambda, const CartesianTwist& twist) {

source/state_representation/src/space/cartesian/CartesianWrench.cpp

@@ -141,7 +141,7 @@ std::ostream& operator<<(std::ostream& os, const CartesianWrench& wrench) {
 }
 
 CartesianWrench operator*(const CartesianState& state, const CartesianWrench& wrench) {
-  return CartesianWrench(state.operator*(wrench));
+  return state.operator*(wrench);
 }
 
 CartesianWrench operator*(double lambda, const CartesianWrench& wrench) {