-
Notifications
You must be signed in to change notification settings - Fork 40
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(autoware_qp_interface): porting autoware_qp_interface package from autoware.universe #146
feat(autoware_qp_interface): porting autoware_qp_interface package from autoware.universe #146
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
6d9c5c9
to
4ae5163
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since it’s a porting task, I’m keeping the review to a minimum.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #146 +/- ##
===========================================
- Coverage 78.75% 61.26% -17.50%
===========================================
Files 11 20 +9
Lines 193 759 +566
Branches 73 412 +339
===========================================
+ Hits 152 465 +313
- Misses 11 137 +126
- Partials 30 157 +127
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…om autoware.universe Signed-off-by: NorahXiong <[email protected]>
155e821
to
b2eae23
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Fetch autoware_qp_interface from autoware.universe.
Related links
Should be handled together with #9824
How was this PR tested?
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.