fix(radar_fusion_to_detected_object): fix knownConditionTrueFalse

[veqcc]

Description

This is a cppcheck warning

perception/radar_fusion_to_detected_object/src/radar_fusion_to_detected_object.cpp:98:32: style: Condition 'split_objects.size()==1' is always true [knownConditionTrueFalse]
      if (split_objects.size() == 1) {
                               ^

What did you mean here?
Maybe my modification split_object.size() == 1 is incorrect.

Related links

Parent Issue:

• Link

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

[github-actions]

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

• You've checked our contribution guidelines.
• Your PR follows our pull request guidelines.
• All required CI checks pass before marking the PR ready for review.

[technolojin]

I think that is also weird.

@scepter914 Can you look at this point?

[scepter914]

The function of splitting objects is not implemented yet.
https://github.com/autowarefoundation/autoware.universe/blob/f7b9180d29713fe177b158ea19d7abb994c12b65/perception/radar_fusion_to_detected_object/src/radar_fusion_to_detected_object.cpp#L89

So the case that always outputs 'split_objects.size()==1' is always true works as expected.
It derive from my first design, but I think this package is not used for product and development of this package is low priority for now.

[veqcc]

@scepter914

So the case that always outputs 'split_objects.size()==1' is always true works as expected.
It derive from my first design, but I think this package is not used for product and development of this package is low priority for now.

OK, then I will add inline suppression for cppcheck.
If you complete your implementation, please remove the inline suppression then 👍

[scepter914]

LGTM

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.55%. Comparing base (fbbc44c) to head (71b7a43).
Report is 18 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7814      +/-   ##
==========================================
- Coverage   28.55%   28.55%   -0.01%     
==========================================
  Files        1586     1586              
  Lines      115879   115888       +9     
  Branches    49384    49387       +3     
==========================================
  Hits        33093    33093              
- Misses      73823    73832       +9     
  Partials     8963     8963              

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <ø> (?)
total	28.55% <ø> (ø)		Carriedforward from fbbc44c

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.