ci: add roscope CI for integrity check#1797
Open
paulsohn wants to merge 15 commits intoautowarefoundation:mainfrom
Open
ci: add roscope CI for integrity check#1797paulsohn wants to merge 15 commits intoautowarefoundation:mainfrom
paulsohn wants to merge 15 commits intoautowarefoundation:mainfrom
Conversation
|
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
d471d97 to
747e727
Compare
paulsohn
commented
Mar 17, 2026
af5da80 to
950f682
Compare
Contributor
Author
|
@mitsudome-r @xmfcx I am drafting this PR for reworks on the tool and reformalization of the problem, but nonetheless I need #1762 to be merged to continue my work. |
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
950f682 to
c3bef5b
Compare
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
Contributor
Author
|
e.g. param file not found issue can be caught with this CI check. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
This is a demonstration of https://github.com/paulsohn/roscope in autoware_launch CI.
It resolves the actual launcher path executed by
ros2 launch autoware_launch autoware.launch.xml sensor_model:=sampe_sensor_kit vehicle_model:=sample_vehicle map_path:=[map_path](map path is just a placeholder now), inlining the includes and evaluating args and variables, to obtain a clear blueprint that what the launcher wants to execute and what packages should be built for it.See https://github.com/orgs/autowarefoundation/discussions/6870 as well: only with roscope locally, I already found 3 bugs and 2 potential defects.
How was this PR tested?
https://github.com/paulsohn/roscope/blob/devel/.github/workflows/autoware-integration.yml
roscope repository runs CI frequently to watch if there is any defect on the entire build system.
This CI check gives an identical experience of the autoware_launch repository itself.
Notes for reviewers
Since the launch system expects ML param files to be in the specified directory, ML parameter files are also added to the PR so they can be copied into the expected directory for the CI check.
ci_depends.reposare excerpted fromrepositories/autoware.reposandrepositories/autoware-nightly.repos, so they should be updated if you want a version change of the launch dependencies (e.g. we recently bumped nebula to v0.3.x).Effects on system behavior
None. This is just a CI check.