Skip to content

Commit

Permalink
Use pose multiplication instead of addition
Browse files Browse the repository at this point in the history
The ign-math Pose addition operator is going to be
deprecated, so use the multiplication operator instead.
It works in the opposite order, matching the behavior
of coordinate transform multiplication.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
  • Loading branch information
scpeters committed Mar 3, 2022
1 parent 0e1f36b commit a3c2475
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/integration/air_pressure.cc
Original file line number Diff line number Diff line change
Expand Up @@ -192,8 +192,8 @@ TEST_F(AirPressureSensorTest, SensorReadings)
EXPECT_DOUBLE_EQ(vertRef, sensor->ReferenceAltitude());
EXPECT_DOUBLE_EQ(vertRef, sensorNoise->ReferenceAltitude());

sensor->SetPose(sensorNoise->Pose() +
ignition::math::Pose3d(0, 0, 1.5, 0, 0, 0));
sensor->SetPose(
ignition::math::Pose3d(0, 0, 1.5, 0, 0, 0) * sensorNoise->Pose());

// verify msg received on the topic
WaitForMessageTestHelper<ignition::msgs::FluidPressure> msgHelper(topic);
Expand Down

0 comments on commit a3c2475

Please sign in to comment.