Skip to content
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

refactor(pure_pursuit): prefix package and namespace with autoware_ #7301

Conversation

zulfaqar-azmi-t4
Copy link
Contributor

@zulfaqar-azmi-t4 zulfaqar-azmi-t4 commented Jun 6, 2024

Description

This PR adds the autoware_ prefix to the package and puts headers in the autoware namespace.

Part of:

Tests performed

Change lateral controller method so that pure pursuit is used.
run psim and ensure that the vehicle can be engaged.

Not applicable.

Effects on system behavior

Not applicable.

Interface changes

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@zulfaqar-azmi-t4 zulfaqar-azmi-t4 changed the title RT1-6683 add autoware prefix to package and namepace refactor(pure_pursuit): prefix package and namespace with autoware_ Jun 6, 2024
@github-actions github-actions bot added type:documentation Creating or refining documentation. (auto-assigned) component:control Vehicle control algorithms and mechanisms. (auto-assigned) labels Jun 6, 2024
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 force-pushed the RT1-6683-autoware-prefix-for-pure-pursuit branch from d69a95f to e9b8e02 Compare June 10, 2024 07:37
@takayuki5168 takayuki5168 added the tag:run-build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Jun 11, 2024
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 force-pushed the RT1-6683-autoware-prefix-for-pure-pursuit branch 3 times, most recently from ef527dd to 92d1125 Compare June 11, 2024 06:08
Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 force-pushed the RT1-6683-autoware-prefix-for-pure-pursuit branch from 637f54e to 90543ca Compare June 11, 2024 09:42
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 enabled auto-merge (squash) June 11, 2024 09:42
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 merged commit 0125e05 into autowarefoundation:main Jun 11, 2024
24 checks passed
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 deleted the RT1-6683-autoware-prefix-for-pure-pursuit branch June 11, 2024 09:45
KhalilSelyan pushed a commit that referenced this pull request Jul 22, 2024
…7301)

* RT1-6683 add autoware prefix to package and namepace

Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>

* fix precommit

Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>

---------

Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:control Vehicle control algorithms and mechanisms. (auto-assigned) tag:run-build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) type:documentation Creating or refining documentation. (auto-assigned)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants