fix(lidar_centerpoint): non-maximum suppression target decision logic #9595
CodeScene Delta Analysis / CodeScene Cloud Delta Analysis (main)
succeeded
Dec 10, 2024 in 36s
CodeScene PR Check
✅ Code Health Quality Gates: OK
Change in average Code Health of affected files: +0.05 (9.32 -> 9.38)
- Declining Code Health: 1 findings(s) 🚩
- Improving Code Health: 4 findings(s) ✅
Details
🚩 Declining Code Health (highest to lowest):
- Complex Conditional non_maximum_suppression.cpp: NonMaximumSuppression::isTargetPairObject
✅ Improving Code Health:
- Large Method node.cpp: LidarCenterPointNode::LidarCenterPointNode
- Large Method test_nms.cpp: TEST:NonMaximumSuppressionTest:Apply
- Bumpy Road Ahead non_maximum_suppression.cpp: NonMaximumSuppression::generateIoUMatrix
- Deep, Nested Complexity non_maximum_suppression.cpp: NonMaximumSuppression::generateIoUMatrix
Annotations
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
❌ New issue: Complex Conditional
NonMaximumSuppression::isTargetPairObject has 1 complex conditionals with 2 branches, threshold = 2. A complex conditional is an expression inside a branch (e.g. if, for, while) which consists of multiple, logical operators such as AND/OR. The more logical operators in an expression, the more severe the code smell.
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
✅ No longer an issue: Bumpy Road Ahead
NonMaximumSuppression::generateIoUMatrix is no longer above the threshold for logical blocks with deeply nested code
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
✅ No longer an issue: Deep, Nested Complexity
NonMaximumSuppression::generateIoUMatrix is no longer above the threshold for nested complexity depth
Check notice on line 82 in perception/autoware_lidar_centerpoint/src/node.cpp
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
✅ Getting better: Large Method
LidarCenterPointNode::LidarCenterPointNode decreases from 90 to 87 lines of code, threshold = 70. Large functions with many lines of code are generally harder to understand and lower the code health. Avoid adding more lines to this function.
Check notice on line 26 in perception/autoware_lidar_centerpoint/test/test_nms.cpp
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
✅ Getting better: Large Method
TEST:NonMaximumSuppressionTest:Apply decreases from 82 to 80 lines of code, threshold = 70. Large functions with many lines of code are generally harder to understand and lower the code health. Avoid adding more lines to this function.
Loading