-
Notifications
You must be signed in to change notification settings - Fork 676
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
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
15844fd
to
0f327a8
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9790 +/- ##
==========================================
+ Coverage 28.77% 29.71% +0.94%
==========================================
Files 1457 1462 +5
Lines 109221 109273 +52
Branches 42566 42767 +201
==========================================
+ Hits 31428 32471 +1043
+ Misses 74714 73680 -1034
- Partials 3079 3122 +43
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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]>
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]>
b0f521f
to
d9a331d
Compare
* feat(behavior_velocity_crosswalk): add node test Signed-off-by: Takayuki Murooka <[email protected]> * fix Signed-off-by: Takayuki Murooka <[email protected]> * feat(behavior_velocity_xxx_module): add node test Signed-off-by: Takayuki Murooka <[email protected]> * fix Signed-off-by: Takayuki Murooka <[email protected]> * fix Signed-off-by: Takayuki Murooka <[email protected]> * fix Signed-off-by: Takayuki Murooka <[email protected]> * fix Signed-off-by: Takayuki Murooka <[email protected]> * change directory tests -> test Signed-off-by: Takayuki Murooka <[email protected]> --------- Signed-off-by: Takayuki Murooka <[email protected]>
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.