Skip to content

Commit d434e57

Browse files
feat: jazzy support (#276)
* feat: jazzy support Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * ci(pre-commit): autofix * fix: cspell Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * docs: adjust dead links Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * refactor: adjust filename Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * feat: keep humble distro compatibility Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * ci(pre-commit): autofix * feat: remove outdated sync Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * feat(calibrators): adjust deprecated matplotlib API Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * chore: update & restore workflows Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * style: apply pre-commit Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * fix(intrinsic_camera_calibrator): pop-up window with matplotlib Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * chore: restore upstream Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent 5dcc5b1 commit d434e57

File tree

70 files changed

+2366
-144
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

70 files changed

+2366
-144
lines changed

.clang-format

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,14 @@
1+
# This file is automatically synced from:
2+
# https://github.com/autowarefoundation/sync-file-templates
3+
# To make changes, update the source repository and follow the guidelines in its README.
4+
15
# Modified from https://github.com/ament/ament_lint/blob/master/ament_clang_format/ament_clang_format/configuration/.clang-format
26
Language: Cpp
37
BasedOnStyle: Google
48

59
AccessModifierOffset: -2
610
AlignAfterOpenBracket: AlwaysBreak
11+
AllowShortFunctionsOnASingleLine: InlineOnly
712
BraceWrapping:
813
AfterClass: true
914
AfterFunction: true
@@ -33,6 +38,9 @@ IncludeCategories:
3338
- Regex: .*_msgs/.*
3439
Priority: 3
3540
CaseSensitive: true
41+
- Regex: .*_srvs/.*
42+
Priority: 3
43+
CaseSensitive: true
3644
# Other Package headers
3745
- Regex: <.*>
3846
Priority: 2

.cspell.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@
9090
"solvepnp",
9191
"sqpnp",
9292
"srvs",
93+
"stringop",
9394
"subsampled",
9495
"subsamples",
9596
"subsampling",

.github/dependabot.yaml

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,15 @@
1+
# This file is automatically synced from:
2+
# https://github.com/autowarefoundation/sync-file-templates
3+
# To make changes, update the source repository and follow the guidelines in its README.
4+
15
version: 2
26
updates:
37
- package-ecosystem: github-actions
48
directory: /
9+
# https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#scheduleinterval
510
schedule:
6-
interval: daily
11+
interval: monthly
712
open-pull-requests-limit: 1
813
labels:
9-
- bot
10-
- github-actions
14+
- tag:bot
15+
- type:github-actions

.github/sync-files.yaml

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,32 @@
1-
- repository: autowarefoundation/autoware
1+
- repository: autowarefoundation/sync-file-templates
22
files:
33
- source: .github/dependabot.yaml
4+
- source: .github/workflows/build-and-test.yaml
5+
- source: .github/workflows/build-and-test-differential.yaml
6+
- source: .github/workflows/cancel-previous-workflows.yaml
7+
- source: .github/workflows/check-build-depends.yaml
8+
- source: .github/workflows/clang-tidy-pr-comments.yaml
9+
- source: .github/workflows/clang-tidy-pr-comments-manually.yaml
410
- source: .github/workflows/pre-commit.yaml
511
- source: .github/workflows/pre-commit-optional.yaml
612
- source: .github/workflows/semantic-pull-request.yaml
713
- source: .github/workflows/spell-check-differential.yaml
14+
pre-commands: |
15+
sd " with:\n" " with:\n local-cspell-json: .cspell.json\n" {source}
16+
- source: .github/workflows/spell-check-daily.yaml
17+
pre-commands: |
18+
sd " with:\n" " with:\n local-cspell-json: .cspell.json\n" {source}
19+
- source: .github/workflows/sync-files.yaml
820
- source: .clang-format
921
- source: .markdown-link-check.json
1022
- source: .markdownlint.yaml
1123
- source: .pre-commit-config-optional.yaml
24+
- source: .pre-commit-config.yaml
1225
- source: .prettierignore
1326
- source: .prettierrc.yaml
1427
- source: .yamllint.yaml
28+
- source: CODE_OF_CONDUCT.md
29+
- source: CONTRIBUTING.md
1530
- source: CPPLINT.cfg
31+
- source: LICENSE
1632
- source: setup.cfg
17-
18-
- repository: autowarefoundation/autoware_common
19-
files:
20-
- source: .github/workflows/build-and-test.yaml
21-
- source: .github/workflows/build-and-test-differential.yaml
22-
- source: .github/workflows/check-build-depends.yaml
23-
- source: .github/workflows/clang-tidy-pr-comments.yaml
24-
- source: .github/workflows/sync-files.yaml

.github/workflows/build-and-test-differential.yaml

Lines changed: 36 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,57 @@
1+
# This file is automatically synced from:
2+
# https://github.com/autowarefoundation/sync-file-templates
3+
# To make changes, update the source repository and follow the guidelines in its README.
4+
15
name: build-and-test-differential
26

37
on:
48
pull_request:
9+
types:
10+
- opened
11+
- synchronize
12+
- reopened
13+
- labeled
14+
15+
concurrency:
16+
group: ${{ github.workflow }}-${{ github.event_name }}-${{ github.ref }}
17+
cancel-in-progress: true
518

619
jobs:
20+
require-label:
21+
uses: autowarefoundation/autoware-github-actions/.github/workflows/require-label.yaml@v1
22+
with:
23+
label: run:build-and-test-differential
24+
725
build-and-test-differential:
8-
runs-on: ubuntu-latest
26+
needs: require-label
27+
if: ${{ needs.require-label.outputs.result == 'true' }}
28+
runs-on: ubuntu-22.04
929
container: ${{ matrix.container }}
1030
strategy:
1131
fail-fast: false
1232
matrix:
1333
rosdistro:
1434
- humble
35+
- jazzy
1536
include:
1637
- rosdistro: humble
1738
container: ros:humble
1839
build-depends-repos: build_depends.repos
40+
- rosdistro: jazzy
41+
container: ros:jazzy
42+
build-depends-repos: build_depends.repos
1943
steps:
20-
- name: Cancel previous runs
21-
uses: styfle/cancel-workflow-action@0.12.1
44+
- name: Set PR fetch depth
45+
run: echo "PR_FETCH_DEPTH=$(( ${{ github.event.pull_request.commits }} + 1 ))" >> "${GITHUB_ENV}"
2246

23-
- name: Check out repository
47+
- name: Checkout PR branch and all PR commits
2448
uses: actions/checkout@v4
2549
with:
26-
fetch-depth: 0
50+
ref: ${{ github.event.pull_request.head.sha }}
51+
fetch-depth: ${{ env.PR_FETCH_DEPTH }}
52+
53+
- name: Show disk space before the tasks
54+
run: df -h
2755

2856
- name: Remove exec_depend
2957
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1
@@ -57,3 +85,6 @@ jobs:
5785
fail_ci_if_error: false
5886
verbose: true
5987
flags: differential
88+
89+
- name: Show disk space after the tasks
90+
run: df -h

.github/workflows/build-and-test.yaml

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
# This file is automatically synced from:
2+
# https://github.com/autowarefoundation/sync-file-templates
3+
# To make changes, update the source repository and follow the guidelines in its README.
4+
15
name: build-and-test
26

37
on:
@@ -6,23 +10,36 @@ on:
610
- cron: 0 0 * * *
711
workflow_dispatch:
812

13+
concurrency:
14+
group: ${{ github.workflow }}-${{ github.event_name }}-${{ github.ref }}
15+
cancel-in-progress: true
16+
917
jobs:
1018
build-and-test:
1119
if: ${{ github.event_name != 'push' || github.ref_name == github.event.repository.default_branch }}
12-
runs-on: ubuntu-latest
20+
runs-on: ubuntu-22.04
1321
container: ${{ matrix.container }}
1422
strategy:
1523
fail-fast: false
1624
matrix:
1725
rosdistro:
1826
- humble
27+
- jazzy
1928
include:
2029
- rosdistro: humble
2130
container: ros:humble
2231
build-depends-repos: build_depends.repos
32+
- rosdistro: jazzy
33+
container: ros:jazzy
34+
build-depends-repos: build_depends.repos
2335
steps:
2436
- name: Check out repository
2537
uses: actions/checkout@v4
38+
with:
39+
fetch-depth: 1
40+
41+
- name: Show disk space before the tasks
42+
run: df -h
2643

2744
- name: Remove exec_depend
2845
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1
@@ -56,3 +73,6 @@ jobs:
5673
fail_ci_if_error: false
5774
verbose: true
5875
flags: total
76+
77+
- name: Show disk space after the tasks
78+
run: df -h
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# This file is automatically synced from:
2+
# https://github.com/autowarefoundation/sync-file-templates
3+
# To make changes, update the source repository and follow the guidelines in its README.
4+
5+
# Deprecated: Use concurrency in each workflow instead.
6+
7+
name: cancel-previous-workflows
8+
9+
on:
10+
pull_request_target:
11+
12+
jobs:
13+
cancel-previous-workflows:
14+
runs-on: ubuntu-22.04
15+
steps:
16+
- name: Cancel previous runs
17+
uses: styfle/cancel-workflow-action@0.12.1
18+
with:
19+
workflow_id: all
20+
all_but_latest: true

.github/workflows/check-build-depends.yaml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,21 @@
1+
# This file is automatically synced from:
2+
# https://github.com/autowarefoundation/sync-file-templates
3+
# To make changes, update the source repository and follow the guidelines in its README.
4+
15
name: check-build-depends
26

37
on:
48
pull_request:
59
paths:
610
- build_depends*.repos
711

12+
concurrency:
13+
group: ${{ github.workflow }}-${{ github.event_name }}-${{ github.ref }}
14+
cancel-in-progress: true
15+
816
jobs:
917
check-build-depends:
10-
runs-on: ubuntu-latest
18+
runs-on: ubuntu-22.04
1119
container: ${{ matrix.container }}
1220
strategy:
1321
fail-fast: false
Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,71 @@
1+
# This file is automatically synced from:
2+
# https://github.com/autowarefoundation/sync-file-templates
3+
# To make changes, update the source repository and follow the guidelines in its README.
4+
5+
name: clang-tidy-pr-comments-manually
6+
7+
on:
8+
workflow_dispatch:
9+
inputs:
10+
workflow_run_id_or_url:
11+
description: The target workflow run ID or URL of the build-and-test-differential workflow
12+
required: true
13+
14+
concurrency:
15+
group: ${{ github.workflow }}-${{ github.event_name }}-${{ github.ref }}
16+
cancel-in-progress: true
17+
18+
jobs:
19+
clang-tidy-pr-comments-manually:
20+
runs-on: ubuntu-22.04
21+
steps:
22+
- name: Check out repository
23+
uses: actions/checkout@v4
24+
25+
- name: Download analysis results
26+
run: |
27+
workflow_run_id=$(echo "${{ inputs.workflow_run_id_or_url }}" | sed -e "s|.*runs/||" -e "s|/jobs.*||")
28+
gh run download "$workflow_run_id" -D /tmp || true
29+
env:
30+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
31+
32+
- name: Check if the fixes.yaml file exists
33+
id: check-fixes-yaml-existence
34+
uses: autowarefoundation/autoware-github-actions/check-file-existence@v1
35+
with:
36+
files: /tmp/clang-tidy-result/fixes.yaml
37+
38+
- name: Set variables
39+
if: ${{ steps.check-fixes-yaml-existence.outputs.exists == 'true' }}
40+
id: set-variables
41+
run: |
42+
echo "pr-id=$(cat /tmp/clang-tidy-result/pr-id.txt)" >> $GITHUB_OUTPUT
43+
echo "pr-head-repo=$(cat /tmp/clang-tidy-result/pr-head-repo.txt)" >> $GITHUB_OUTPUT
44+
echo "pr-head-ref=$(cat /tmp/clang-tidy-result/pr-head-ref.txt)" >> $GITHUB_OUTPUT
45+
46+
- name: Check out PR head
47+
if: ${{ steps.check-fixes-yaml-existence.outputs.exists == 'true' }}
48+
uses: actions/checkout@v4
49+
with:
50+
repository: ${{ steps.set-variables.outputs.pr-head-repo }}
51+
ref: ${{ steps.set-variables.outputs.pr-head-ref }}
52+
persist-credentials: false
53+
54+
- name: Replace paths in fixes.yaml
55+
if: ${{ steps.check-fixes-yaml-existence.outputs.exists == 'true' }}
56+
run: |
57+
sed -i -e "s|/__w/|/home/runner/work/|g" /tmp/clang-tidy-result/fixes.yaml
58+
cat /tmp/clang-tidy-result/fixes.yaml
59+
60+
- name: Copy fixes.yaml to access from Docker Container Action
61+
if: ${{ steps.check-fixes-yaml-existence.outputs.exists == 'true' }}
62+
run: |
63+
cp /tmp/clang-tidy-result/fixes.yaml fixes.yaml
64+
65+
- name: Run clang-tidy-pr-comments action
66+
if: ${{ steps.check-fixes-yaml-existence.outputs.exists == 'true' }}
67+
uses: platisd/clang-tidy-pr-comments@v1
68+
with:
69+
github_token: ${{ secrets.GITHUB_TOKEN }}
70+
clang_tidy_fixes: fixes.yaml
71+
pull_request_id: ${{ steps.set-variables.outputs.pr-id }}

0 commit comments

Comments
 (0)