Skip to content

Commit

Permalink
#2215 Use actual covariance matrix
Browse files Browse the repository at this point in the history
The multiple_object_tracker_component was not using incoming detections'
covariance matrices. Instead, it was using the zero matrix. The node
now uses the detections' covariances.
  • Loading branch information
adamlm committed Dec 15, 2023
1 parent a3d27f6 commit 677ec58
Showing 1 changed file with 13 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,12 @@ auto make_ctrv_detection(
mot::Angle{units::angle::radian_t{yaw}},
units::angular_velocity::radians_per_second_t{msg.twist.twist.angular.z}};

const mot::CtrvStateCovariance covariance = mot::CtrvStateCovariance::Zero();
mot::CtrvStateCovariance covariance = mot::CtrvStateCovariance::Zero();
covariance(0, 0) = msg.pose.covariance.at(0);
covariance(1, 1) = msg.pose.covariance.at(7);
covariance(2, 2) = msg.twist.covariance.at(0);
covariance(3, 3) = msg.pose.covariance.at(35);
covariance(4, 4) = msg.twist.covariance.at(35);

return mot::CtrvDetection{timestamp, state, covariance, mot::Uuid{msg.id}};
}
Expand Down Expand Up @@ -96,7 +101,13 @@ auto make_ctra_detection(
units::angular_velocity::radians_per_second_t{msg.twist.twist.angular.z},
units::acceleration::meters_per_second_squared_t{msg.accel.accel.linear.x}};

const mot::CtraStateCovariance covariance = mot::CtraStateCovariance::Zero();
mot::CtraStateCovariance covariance = mot::CtraStateCovariance::Zero();
covariance(0, 0) = msg.pose.covariance.at(0);
covariance(1, 1) = msg.pose.covariance.at(7);
covariance(2, 2) = msg.twist.covariance.at(0);
covariance(3, 3) = msg.pose.covariance.at(35);
covariance(4, 4) = msg.twist.covariance.at(35);
covariance(5, 5) = msg.accel.covariance.at(0);

return mot::CtraDetection{timestamp, state, covariance, mot::Uuid{msg.id}};
}
Expand Down

0 comments on commit 677ec58

Please sign in to comment.