feat(srr520): initial driver

[knzo25]

PR Type

• New Feature

Related Links

Description

Initial hardware interface, decoder, and ros wrappers for the continental SRR520

Review Procedure

Remarks

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

• Assign PR to reviewer

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

• Commits are properly organized and messages are according to the guideline
• (Optional) Unit tests have been written for new behavior
• PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

• All open points are addressed and tracked via issues or tickets

CI Checks

• Build and test for PR: Required to pass before the merge.

[knzo25]

@mojomex
Although it is marked for review, we still need to merge the ARS548 PR before addressing this one (also, the priority for this one is quite low)

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 20.13559% with 1178 lines in your changes missing coverage. Please review.

Please upload report for BASE (develop@44dea6f). Learn more about missing BASE report.

Files	Patch %	Lines
...ontinental/decoders/continental_srr520_decoder.cpp	43.82%	135 Missing and 188 partials ⚠️
...continental/continental_srr520_decoder_wrapper.cpp	0.00%	313 Missing ⚠️
..._hw_interfaces/continental_srr520_hw_interface.cpp	0.00%	202 Missing ⚠️
...src/continental/continental_srr520_ros_wrapper.cpp	0.00%	101 Missing ⚠️
...ontinental/continental_ros_decoder_test_srr520.cpp	29.41%	3 Missing and 81 partials ⚠️
...nental/continental_srr520_hw_interface_wrapper.cpp	0.00%	82 Missing ⚠️
...src/continental/continental_ars548_ros_wrapper.cpp	0.00%	33 Missing ⚠️
...ontinental/continental_ros_decoder_test_ars548.cpp	0.00%	0 Missing and 17 partials ⚠️
...ental/continental_ros_decoder_test_main_srr520.cpp	36.84%	1 Missing and 11 partials ⚠️
nebula_tests/continental/parameter_descriptors.cpp	12.50%	6 Missing and 1 partial ⚠️
... and 3 more

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             develop     #124   +/-   ##
==========================================
  Coverage           ?   15.11%           
==========================================
  Files              ?       93           
  Lines              ?     8888           
  Branches           ?     1549           
==========================================
  Hits               ?     1343           
  Misses             ?     6453           
  Partials           ?     1092           

Flag	Coverage Δ
differential	15.11% <20.13%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[knzo25]

This PR should be reviewed after #151 is merged

[mojomex]

Apart from the feedback left above, everything looks good to me!

[mojomex]

LGTM!