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

Remove tier4_ad_api_adaptor #3096

Open
3 tasks done
isamu-takagi opened this issue Dec 1, 2022 · 6 comments · Fixed by autowarefoundation/autoware-documentation#276
Open
3 tasks done

Remove tier4_ad_api_adaptor #3096

isamu-takagi opened this issue Dec 1, 2022 · 6 comments · Fixed by autowarefoundation/autoware-documentation#276
Assignees
Labels
component:interfaces Communication protocols, interfaces and messages. type:new-feature New functionalities or additions, feature requests.

Comments

@isamu-takagi
Copy link
Contributor

isamu-takagi commented Dec 1, 2022

Checklist

  • I've read the contribution guidelines.
  • I've searched other issues and no duplicate issues were found.
  • I've agreed with the maintainers that I can plan this task.

Description

Currently, tier4_ad_api_adaptor is imported as a sample API. However, it is now possible to drive the vehicle even with the newly introduced AD API. It's confusing to have two API examples, so I plan to remove tier4_ad_api_adaptor.

Purpose

Unify API samples to default_ad_api.

Possible approaches

  1. Replace code that uses tier4_ad_api_adaptor with default_ad_api.
  2. Remove tier4_ad_api_adaptor from autoware.repos.

Definition of done

Remove tier4_ad_api_adaptor from autoware.repos

@xmfcx
Copy link
Contributor

xmfcx commented Nov 16, 2023

Hello @isamu-takagi -san,

autoware/autoware.repos

Lines 28 to 31 in c9c4546

universe/external/tier4_ad_api_adaptor: # TODO(TIER IV): Improve design/code and transfer to AWF
type: git
url: https://github.com/tier4/tier4_ad_api_adaptor.git
version: tier4/universe

Since this comment is present, I guess this issue is still relevant.

Is there a timeline for this issue to be completed? (Is this low priority?)

Could you provide a list of packages to be modified/refactored to achieve this task?

@xmfcx xmfcx added the type:new-feature New functionalities or additions, feature requests. label Nov 16, 2023
@isamu-takagi
Copy link
Contributor Author

@xmfcx RTC has not been deleted yet.

This will be completed by migrating to this API. I'm currently implementing this API.
https://autowarefoundation.github.io/autoware-documentation/main/design/autoware-interfaces/ad-api/features/cooperation/

@xmfcx
Copy link
Contributor

xmfcx commented Nov 17, 2023

@isamu-takagi thanks for the information.

I couldn't find Cooperation API in the default_ad_api.

I'm currently implementing this API.

Is it being implemented into the default_ad_api package?

Are there ongoing PRs for this task?

@isamu-takagi
Copy link
Contributor Author

isamu-takagi commented Nov 17, 2023

There are no PRs for AD API yet. I'm currently changing the implementation to match the API.

@idorobotics idorobotics moved this to In Progress in Autoware Labs Nov 22, 2023
@xmfcx xmfcx removed this from Autoware Labs Mar 18, 2024
@xmfcx
Copy link
Contributor

xmfcx commented Feb 17, 2025

@isamu-takagi -san
I've searched the current Autoware workspace for the usages of this package but couldn't find any. Can we remove tier4_ad_api_adaptor from Autoware completely now?

sudo apt install ripgrep -y

mfc@bunny:~/projects/autoware$ rg "tier4_ad_api_adaptor" --hidden --no-ignore -g '!*.rst' -g '!{build,install,log,**/.git,**/.idea}/**'

./autoware-nightly.repos
22:  universe/external/tier4_ad_api_adaptor: # TODO(TIER IV): Migrate to AD API and remove this repository entry.
24:    url: https://github.com/tier4/tier4_ad_api_adaptor.git

./src/universe/external/tier4_ad_api_adaptor/.github/workflows/sync-micro-bus.yaml
25:          sync-target-repository: https://github.com/tier4/tier4_ad_api_adaptor

./autoware.repos
37:  universe/external/tier4_ad_api_adaptor: # TODO(TIER IV): Migrate to AD API and remove this repository entry.
39:    url: https://github.com/tier4/tier4_ad_api_adaptor.git

@isamu-takagi
Copy link
Contributor Author

isamu-takagi commented Feb 17, 2025

@xmfcx The following packages in tier4_ad_api_adaptor are used when launch_deprecated_api=true. So, it is fine if the option is set to false (default value), but setting it to true will cause an error.

  • autoware_iv_external_api_adaptor
  • autoware_iv_internal_api_adaptor
  • awapi_awiv_adapter

Releted files:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:interfaces Communication protocols, interfaces and messages. type:new-feature New functionalities or additions, feature requests.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants