Use TrackedObjectKinematics instead of DetectedObjectKinematics for TrackedObject & object

[felixf4xu]

Checklist

• I've read the contribution guidelines.
• I've searched other issues and no duplicate issues were found.
• I'm convinced that this is not my fault but a bug.

Description

In autoware, there are 2 kinds of ObjectKinematics definition, one is TrackedObjectKinematics and the other is DetectedObjectKinematics, they are equal to each other in integer value, but they should have difference namespaces.__

Expected behavior

void MapBasedPredictionNode::updateObjectData(TrackedObject & object)
{
  std::unique_ptr<ScopedTimeTrack> st_ptr;
  if (time_keeper_) st_ptr = std::make_unique<ScopedTimeTrack>(__func__, *time_keeper_);


  if (
    object.kinematics.orientation_availability ==
    autoware_perception_msgs::msg::DetectedObjectKinematics::AVAILABLE) {
    return;
  }

DetectedObjectKinematics is a wrong type, it should be TrackedObjectKinematics because of code type TrackedObject & object

for same reason,
code

 switch (object.kinematics.orientation_availability) {
    case autoware_perception_msgs::msg::DetectedObjectKinematics::SIGN_UNKNOWN: {

should also use TrackedObjectKinematics

Actual behavior

DetectedObjectKinematics is used for TrackedObject type.

Steps to reproduce

check source code, code review.

Versions

autoware, latest main branch

Possible causes

No response

Additional context

No response

[amadeuszsz]

@felixf4xu good catch & thank you for your report! Addressed in #9553