feat(api): set launch_deprecated_api true #496
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
https://star4.slack.com/archives/C03QW0GU6P7/p1716891686276349?thread_ts=1715690581.066659&cid=C03QW0GU6P7 を参照。
https://github.com/tier4/autoware_launch/blob/tier4/main/autoware_launch/launch/components/tier4_autoware_api_component.launch.xml#L4 では launch_deprecated_api=trueにしているが、awf-latestではpsimではfalseになる。この場合FOAの起動が出来なくなるなどの不便が生じるため、tier4/mainに合わせてtrueにする。
ローカルでawf/mainのマージが出来てたので、awf/main -> tier4/awf-latestのsync (https://github.com/tier4/autoware_launch/blob/tier4/main/.github/workflows/sync-awf-latest.yaml) も動くはず?
(動かなかったらsync修正する必要あり)
Tests performed
2024/07/18 https://evaluation.tier4.jp/evaluation/reports/dcff9a18-1edc-593d-86e3-65808b3fe2ce/?project_id=prd_jt
psim

foaは一部無効にしないと動かないが別問題そう

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.
After all checkboxes are checked, anyone who has write access can merge the PR.