fix(bytetrack): fix cppcheck warnings

[veqcc]

Description

This is a fix based on cppcheck warnings

perception/bytetrack/lib/src/lapjv.cpp:184:56: style: Parameter 'd' can be declared as pointer to const [constParameterPointer]
uint_t _find_dense(const uint_t n, uint_t lo, cost_t * d, int_t * cols, [[maybe_unused]] int_t * y)
                                                       ^
perception/bytetrack/lib/src/lapjv.cpp:206:25: style: Parameter 'y' can be declared as pointer to const [constParameterPointer]
  int_t * pred, int_t * y, cost_t * v)
                        ^
perception/bytetrack/lib/src/lapjv.cpp:206:37: style: Parameter 'v' can be declared as pointer to const [constParameterPointer]
  int_t * pred, int_t * y, cost_t * v)
                                    ^
perception/bytetrack/lib/src/strack.cpp:120:27: style: Variable 'xyah' is assigned a value that is never used. [unreadVariable]
  std::vector<float> xyah = tlwh_to_xyah(_tlwh_tmp);
                          ^
perception/bytetrack/lib/src/strack.cpp:135:27: style: Variable 'xyah' is assigned a value that is never used. [unreadVariable]
  std::vector<float> xyah = tlwh_to_xyah(new_track.tlwh);
                          ^
perception/bytetrack/lib/src/strack.cpp:159:27: style: Variable 'xyah' is assigned a value that is never used. [unreadVariable]
  std::vector<float> xyah = tlwh_to_xyah(new_track.tlwh);
                          ^
perception/bytetrack/lib/src/utils.cpp:280:35: style: The scope of the variable 'cost_c_extended' can be reduced. [variableScope]
  std::vector<std::vector<float>> cost_c_extended;
                                  ^
perception/bytetrack/lib/src/utils.cpp:193:24: style: Unused variable: ious_tmp [unusedVariable]
    std::vector<float> ious_tmp;
                       ^
perception/bytetrack/src/bytetrack_visualizer_node.cpp:131:15: style: Variable 'feat_obj' can be declared as reference to const [constVariableReference]
  for (auto & feat_obj : rect_msg->feature_objects) {
              ^
perception/bytetrack/src/bytetrack_visualizer_node.cpp:138:15: style: Variable 'dynamic_obj' can be declared as reference to const [constVariableReference]
  for (auto & dynamic_obj : uuid_msg->objects) {
              ^

Tests performed

Not applicable.

Effects on system behavior

Not applicable.

Interface changes

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

[manato]

@veqcc
I appreciate your contribution and am sorry for the delay in the review.
LGTM! I also confirmed the modified bytetrack node works fine on my local environment