-
Notifications
You must be signed in to change notification settings - Fork 664
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(behavior_velocity_modules): add node test #9790
base: main
Are you sure you want to change the base?
feat(behavior_velocity_modules): add node test #9790
Conversation
dab0398
to
c5890d0
Compare
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
c5890d0
to
ec82355
Compare
0d1cd9b
to
db7dbe9
Compare
db7dbe9
to
15844fd
Compare
Signed-off-by: Takayuki Murooka <[email protected]>
Signed-off-by: Takayuki Murooka <[email protected]>
Signed-off-by: Takayuki Murooka <[email protected]>
Signed-off-by: Takayuki Murooka <[email protected]>
15844fd
to
0f327a8
Compare
Signed-off-by: Takayuki Murooka <[email protected]>
Signed-off-by: Takayuki Murooka <[email protected]>
Signed-off-by: Takayuki Murooka <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9790 +/- ##
==========================================
+ Coverage 28.79% 29.79% +1.00%
==========================================
Files 1456 1462 +6
Lines 109093 109166 +73
Branches 42492 42709 +217
==========================================
+ Hits 31409 32525 +1116
+ Misses 74606 73517 -1089
- Partials 3078 3124 +46
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Description
Continuing the following PR, this PR added a node test to each package in the behavior_velocity_planner.
Please see the description of the following PR for the details.
#9760
Additionally, due to the current include directory's structure (
include/autoware/<package-name>
), it is impossible toat the same time since we got an error of failing to find the library.
Note: This is because the include directories for the implementation library and messages/services are different.
That's why this PR removes the service generation, and we decided to use the following external service in the autoware_internal_debug_msgs package.
https://github.com/autowarefoundation/autoware_internal_msgs/blob/main/autoware_internal_debug_msgs/srv/String.srv
Related links
Parent Issue:
How was this PR tested?
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.