test(signal_processing): fix unit test bug

[takayuki5168]

Signed-off-by: Takayuki Murooka takayuki5168@gmail.com

Description

Related to #1959

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.

[takayuki5168]

Changed the same way as

autoware.universe/common/signal_processing/src/lowpass_filter_1d.cpp

Line 19 in 4c739d6

void LowpassFilter1d::reset() { x_ = {}; }

[kenji-miyake]

Thank you for your fixing.
Could I test CI before merging this PR?

[takayuki5168]

@kenji-miyake Of course. I'm not in a hurry.

[codecov]

Codecov Report

Base: 10.44% // Head: 10.34% // Decreases project coverage by -0.10% ⚠️

Coverage data is based on head (4c739d6) compared to base (93cc9bd).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1960      +/-   ##
==========================================
- Coverage   10.44%   10.34%   -0.11%     
==========================================
  Files        1233     1234       +1     
  Lines       88034    88963     +929     
  Branches    20501    20508       +7     
==========================================
+ Hits         9197     9199       +2     
- Misses      69328    70249     +921     
- Partials     9509     9515       +6     

Flag	Coverage Δ		*Carryforward flag
differential	0.78% <0.00%> (?)
total	10.43% <0.00%> (ø)		Carriedforward from 93cc9bd

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

Impacted Files	Coverage Δ
...ssing/include/signal_processing/lowpass_filter.hpp	100.00% <ø> (ø)
...nal_processing/test/src/lowpass_filter_1d_test.cpp	35.71% <0.00%> (ø)
...signal_processing/test/src/lowpass_filter_test.cpp	25.58% <0.00%> (-0.14%)	⬇️
planning/obstacle_stop_planner/src/node.cpp	0.00% <0.00%> (ø)
planning/obstacle_cruise_planner/src/node.cpp	0.00% <0.00%> (ø)
...lanning/obstacle_stop_planner/src/debug_marker.cpp	0.00% <0.00%> (ø)
...ning/obstacle_cruise_planner/src/polygon_utils.cpp	0.00% <0.00%> (ø)
...e_planner/include/obstacle_cruise_planner/node.hpp	0.00% <0.00%> (ø)
...ner/include/obstacle_stop_planner/debug_marker.hpp	0.00% <0.00%> (ø)
... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ambroise-arm]

LGTM, tested locally on arm

[kenji-miyake]

@ambroise-arm Could you review autowarefoundation/autoware-github-actions#185 before merging this PR, please? 🙏

[takayuki5168]

@kenji-miyake I tested locally with RelWithDebInfo, the issue did not occur.

[kenji-miyake]

Since I was able to reproduce the issue in #1965, it's okay to merge this PR.

[kenji-miyake]

@takayuki5168 Since this PR won't fix any feature, I've changed the title from fix to test.

[takayuki5168]

@kenji-miyake
Okay, but in my understanding, this PR resolved the issue that unit test failed on Arm. Even in this case, Should we set test instead of fix?
#1959

[kenji-miyake]

@takayuki5168 Uh, sorry. I overlooked the first file. 🥺