-
Notifications
You must be signed in to change notification settings - Fork 58
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(trajectory): add API description, nomenclature, illustration, rename functions to align with nomenclature #292
base: main
Are you sure you want to change the base?
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
d189a93
to
3b383b3
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #292 +/- ##
===========================================
- Coverage 78.75% 32.24% -46.52%
===========================================
Files 11 182 +171
Lines 193 11406 +11213
Branches 73 4238 +4165
===========================================
+ Hits 152 3678 +3526
- Misses 11 7242 +7231
- Partials 30 486 +456
☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
3b383b3
to
c66a039
Compare
33352f0
to
c70c4e6
Compare
d185f0e
to
c6f7878
Compare
…name functions to align with nomenclature Signed-off-by: Mamoru Sobue <mamoru.sobue@tier4.jp>
dfa217c
to
c250ead
Compare
Description
Read this : https://autowarefoundation.github.io/autoware_core/pr-292/common/autoware_trajectory/
get_internal_base
toget_underlying_base
to align with nomenclatureRelated links
Parent Issue:
How was this PR tested?
unit test passes
Notes for reviewers
None.
Interface changes
add following member functions to trajectory_class. They will be 100% used when planning node starts using this class
compute(vector<double>)
curvature(vector<double>)
Effects on system behavior
None.