Skip to content

Commit

Permalink
Merge branch 'main' into feat/add-route-following-check-option
Browse files Browse the repository at this point in the history
  • Loading branch information
isamu-takagi committed Aug 29, 2024
2 parents af69025 + 5e36db1 commit 833bbf3
Show file tree
Hide file tree
Showing 12 changed files with 256 additions and 51 deletions.
4 changes: 4 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE/small-change.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ Not applicable.

Not applicable.

## Interface changes

<!-- Describe any changed interfaces, such as topics, services, or parameters, including debugging interfaces -->

## Pre-review checklist for the PR author

The PR author **must** check the checkboxes below when creating the PR.
Expand Down
13 changes: 13 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE/standard-change.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,19 @@

<!-- Describe any changed interfaces, such as topics, services, or parameters. -->

### ROS Topic Changes

<!-- | Topic Name | Type | Direction | Update Description | -->
<!-- | ---------------- | ------------------- | --------- | ------------------------------------------------------------- | -->
<!-- | `/example_topic` | `std_msgs/String` | Subscribe | Description of what the topic is used for in the system | -->
<!-- | `/another_topic` | `sensor_msgs/Image` | Publish | Also explain if it is added / modified / deleted with the PR | -->

### ROS Parameter Changes

<!-- | Parameter Name | Default Value | Update Description | -->
<!-- | -------------------- | ------------- | --------------------------------------------------- | -->
<!-- | `example_parameters` | `1.0` | Describe the parameter and also explain the updates | -->

## Effects on system behavior

<!-- Describe how this PR affects the system behavior. -->
Expand Down
60 changes: 24 additions & 36 deletions .github/workflows/build-and-test-differential.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,21 @@ name: build-and-test-differential

on:
pull_request:
types:
- opened
- synchronize
- reopened
- labeled

jobs:
make-sure-label-is-present:
uses: autowarefoundation/autoware-github-actions/.github/workflows/make-sure-label-is-present.yaml@v1
with:
label: tag:run-build-and-test-differential

build-and-test-differential:
needs: make-sure-label-is-present
if: ${{ needs.make-sure-label-is-present.outputs.result == 'true' }}
runs-on: ubuntu-latest
container: ${{ matrix.container }}
strategy:
Expand All @@ -16,10 +28,17 @@ jobs:
- rosdistro: humble
container: ros:humble
steps:
- name: Check out repository
- name: Set PR fetch depth
run: echo "PR_FETCH_DEPTH=$(( ${{ github.event.pull_request.commits }} + 1 ))" >> "${GITHUB_ENV}"

- name: Checkout PR branch and all PR commits
uses: actions/checkout@v4
with:
fetch-depth: 0
ref: ${{ github.event.pull_request.head.sha }}
fetch-depth: ${{ env.PR_FETCH_DEPTH }}

- name: Show disk space before the tasks
run: df -h

- name: Remove exec_depend
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1
Expand All @@ -45,43 +64,12 @@ jobs:

- name: Upload coverage to CodeCov
if: ${{ steps.test.outputs.coverage-report-files != '' }}
uses: codecov/codecov-action@v3
uses: codecov/codecov-action@v4
with:
files: ${{ steps.test.outputs.coverage-report-files }}
fail_ci_if_error: false
verbose: true
flags: differential

clang-tidy-differential:
runs-on: ubuntu-latest
container: ros:humble
needs: build-and-test-differential
steps:
- name: Check out repository
uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Remove exec_depend
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1

- name: Get modified packages
id: get-modified-packages
uses: autowarefoundation/autoware-github-actions/get-modified-packages@v1

- name: Get modified files
id: get-modified-files
uses: tj-actions/changed-files@v42
with:
files: |
**/*.cpp
**/*.hpp
- name: Run clang-tidy
if: ${{ steps.get-modified-files.outputs.all_changed_files != '' }}
uses: autowarefoundation/autoware-github-actions/clang-tidy@v1
with:
rosdistro: humble
target-packages: ${{ steps.get-modified-packages.outputs.modified-packages }}
target-files: ${{ steps.get-modified-files.outputs.all_changed_files }}
clang-tidy-config-url: https://raw.githubusercontent.com/autowarefoundation/autoware/main/.clang-tidy
- name: Show disk space after the tasks
run: df -h
16 changes: 8 additions & 8 deletions .github/workflows/build-and-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,14 +22,11 @@ jobs:
steps:
- name: Check out repository
uses: actions/checkout@v4

- name: Free disk space (Ubuntu)
uses: jlumbroso/[email protected]
with:
tool-cache: false
dotnet: false
swap-storage: false
large-packages: false
fetch-depth: 1

- name: Show disk space before the tasks
run: df -h

- name: Remove exec_depend
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1
Expand All @@ -55,9 +52,12 @@ jobs:

- name: Upload coverage to CodeCov
if: ${{ steps.test.outputs.coverage-report-files != '' }}
uses: codecov/codecov-action@v3
uses: codecov/codecov-action@v4
with:
files: ${{ steps.test.outputs.coverage-report-files }}
fail_ci_if_error: false
verbose: true
flags: total

- name: Show disk space after the tasks
run: df -h
1 change: 1 addition & 0 deletions .markdownlint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,6 @@ MD029:
style: ordered
MD033: false
MD041: false
MD045: false
MD046: false
MD049: false
2 changes: 1 addition & 1 deletion .pre-commit-config-optional.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
repos:
- repo: https://github.com/tcort/markdown-link-check
rev: v3.11.2
rev: v3.12.2
hooks:
- id: markdown-link-check
args: [--quiet, --config=.markdown-link-check.json]
6 changes: 3 additions & 3 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ ci:

repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.5.0
rev: v4.6.0
hooks:
- id: check-json
- id: check-merge-conflict
Expand All @@ -18,7 +18,7 @@ repos:
args: [--markdown-linebreak-ext=md]

- repo: https://github.com/igorshubovych/markdownlint-cli
rev: v0.39.0
rev: v0.41.0
hooks:
- id: markdownlint
args: [-c, .markdownlint.yaml, --fix]
Expand All @@ -34,7 +34,7 @@ repos:
- id: yamllint

- repo: https://github.com/tier4/pre-commit-hooks-ros
rev: v0.8.0
rev: v0.10.0
hooks:
- id: prettier-package-xml
- id: sort-package-xml
Expand Down
170 changes: 170 additions & 0 deletions autoware_adapi_v1_msgs/CHANGELOG.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,170 @@
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Changelog for package autoware_adapi_v1_msgs
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

1.3.0 (2024-07-03)
------------------
* feat(autoware_adapi_v1_msgs): remove energy status (`#58 <https://github.com/youtalk/autoware_adapi_msgs/issues/58>`_)
* feat(autoware_adapi_v1_msgs): add diagnostics (`#54 <https://github.com/youtalk/autoware_adapi_msgs/issues/54>`_)
* Contributors: Takagi, Isamu

1.2.1 (2024-04-30)
------------------
* build: set package versions to the package.xml files (`#55 <https://github.com/youtalk/autoware_adapi_msgs/issues/55>`_)
* version 1.1.0
* separate to build\_ and exec\_ depends
* update description
* style(pre-commit): autofix
* Update autoware_adapi_v1_msgs/package.xml
Co-authored-by: Takagi, Isamu <[email protected]>
* Update autoware_adapi_v1_msgs/package.xml
Co-authored-by: Takagi, Isamu <[email protected]>
* Update autoware_adapi_version_msgs/package.xml
Co-authored-by: Takagi, Isamu <[email protected]>
* version 1.2.0
* Revert "separate to build\_ and exec\_ depends"
This reverts commit 6b657cf84a674a8272fbc6571a0f3dffa0a1dce8.
---------
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Takagi, Isamu <[email protected]>
* feat(autoware_adapi_v1_msgs): add heartbeat (`#50 <https://github.com/youtalk/autoware_adapi_msgs/issues/50>`_)
* feat(autoware_adapi_v1_msgs): add heartbeat
* feat(autoware_adapi_v1_msgs): add message counter
* Update autoware_adapi_v1_msgs/system/msg/Heartbeat.msg
Co-authored-by: M. Fatih Cırıt <[email protected]>
* feat: add comment
---------
Co-authored-by: M. Fatih Cırıt <[email protected]>
* feat(autoware_adapi_v1_msgs): remove planning factor type (`#48 <https://github.com/youtalk/autoware_adapi_msgs/issues/48>`_)
* feat(autoware_adapi_v1_msgs): add planning behavior type (`#45 <https://github.com/youtalk/autoware_adapi_msgs/issues/45>`_)
* feat(autoware_adapi_v1_msgs): add MRM pull over behavior for mrm state (`#43 <https://github.com/youtalk/autoware_adapi_msgs/issues/43>`_)
* add pull over behavior to mrm state
* Update autoware_adapi_v1_msgs/system/msg/MrmState.msg
Co-authored-by: Kotaro Yoshimoto <[email protected]>
---------
Co-authored-by: Kotaro Yoshimoto <[email protected]>
* feat(autoware_adapi_v1_msgs): add cooperation (`#41 <https://github.com/youtalk/autoware_adapi_msgs/issues/41>`_)
* feat: add cooperation messages
* feat: add cooperation service
* feat: merge planning factors
* feat: modify constant names
* feat: add cooperation default
* update fields
* upfate default decision service
* rename service
* update message name
* fix constants
* add constants for behavior and sequence
* update field
* fix order
* add comment
---------
* feat(autoware_adapi_v1_msgs): add door command (`#40 <https://github.com/youtalk/autoware_adapi_msgs/issues/40>`_)
* door command
* remove unused constant
* modify command
---------
* feat(autoware_adapi_v1_msgs): add vehicle status msgs (`#24 <https://github.com/youtalk/autoware_adapi_msgs/issues/24>`_)
* add vehicle status msgs
* simplify door msg
* add vehicle state
* change vehicle motion
* clean up
* fix typo
* change geo position to array and remove GeoPosition msg
* update msgs
* update msgs
* update naming
* add none
* update msg
* fix naming
* fix typo
* change naming
* update geopoint to use geographic_msgs
* add commend document
* update message
* Update autoware_adapi_v1_msgs/vehicle/msg/Kinematic.msg
Fix speelling
Co-authored-by: Takagi, Isamu <[email protected]>
* update message name
* update door layout
* Update autoware_adapi_v1_msgs/vehicle/msg/VehicleKinematics.msg
Update commend in VehicleKinematics
Co-authored-by: Ryohsuke Mitsudome <[email protected]>
* style(pre-commit): autofix
---------
Co-authored-by: Takagi, Isamu <[email protected]>
Co-authored-by: Takagi, Isamu <[email protected]>
Co-authored-by: Ryohsuke Mitsudome <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* refactor(start_planner): rename pull out to start planner (`#36 <https://github.com/youtalk/autoware_adapi_msgs/issues/36>`_)
* feat(autoware_adapi_v1_msgs): change door field names to suit (`#35 <https://github.com/youtalk/autoware_adapi_msgs/issues/35>`_)
* feat: first draft proposal implementation for handling invalid lanelets (`#27 <https://github.com/youtalk/autoware_adapi_msgs/issues/27>`_)
* feat: first draft proposal implementation for handling invalid lanelets
* feat: changing module name from invalid_lanelet to no_drivable_lane
---------
* feat(autoware_adapi_v1_msgs): add object recognition msg (`#25 <https://github.com/youtalk/autoware_adapi_msgs/issues/25>`_)
* add perception message for object recognition
* use the correct msg
* change naming
* fix naming
* change shape
* add missing data
* change naming
* change naming
* change shape type
---------
* feat(autoware_adapi_v1_msgs): add vehicle info msgs (`#28 <https://github.com/youtalk/autoware_adapi_msgs/issues/28>`_)
* feat(autoware_adapi_v1_msgs): add vehicle info messages
* feat: add door group
* feat: change door group
* feat: add footprint to vehicle dimensions
---------
* refactor(behavior_path_planner): rename pull_over to goal_planner (`#33 <https://github.com/youtalk/autoware_adapi_msgs/issues/33>`_)
* refactor(behavior_path_planner): rename pull_over to goal_planner
* Update autoware_adapi_v1_msgs/planning/msg/SteeringFactor.msg
Co-authored-by: Takagi, Isamu <[email protected]>
---------
Co-authored-by: Takagi, Isamu <[email protected]>
* feat(autoware_adapi_v1_msgs): add error code for route points (`#32 <https://github.com/youtalk/autoware_adapi_msgs/issues/32>`_)
* feat(autoware_adapi_v1_msgs): add new error messages (`#31 <https://github.com/youtalk/autoware_adapi_msgs/issues/31>`_)
* feat(autoware_adapi_v1_msgs): add new error messages
* Update autoware_adapi_v1_msgs/routing/srv/SetRoute.srv
Co-authored-by: Takagi, Isamu <[email protected]>
---------
Co-authored-by: Takagi, Isamu <[email protected]>
* feat(autoware_adapi_v1_msgs): add routing option for goal modification (`#30 <https://github.com/youtalk/autoware_adapi_msgs/issues/30>`_)
* feat(autoware_adapi_v1_msgs): add goal modification
* feat: change goal modification option
---------
* feat(autoware_adapi_v1_msgs): add msgs for MRM (`#16 <https://github.com/youtalk/autoware_adapi_msgs/issues/16>`_)
* feat(autoware_adapi_v1_msgs): add mrm msgs
* fix(autoware_adapi_v1_msgs): remove mrm behavior status msg
* fix(autoware_adapi_v1_msgs): fix acronyms case
Co-authored-by: Kenji Miyake <[email protected]>
* chore(autoware_adapi_v1_msgs): add comments for states and behaviors
Co-authored-by: Kenji Miyake <[email protected]>
* ci(pre-commit): autofix
* feat(autoware_adapi_v1_msgs): add operation mode api message (`#17 <https://github.com/youtalk/autoware_adapi_msgs/issues/17>`_)
* feat(autoware_adapi_v1_msgs): add operation mode api message
* feat: use minimum size type
* fix(autoware_adapi_v1_msgs): fix file names
* fix(autoware_adapi_v1_msgs): fix constant values
Co-authored-by: Takagi, Isamu <[email protected]>
* remove(autoware_adapi_v1_msgs): remove mrm operation service
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Takagi, Isamu <[email protected]>
* feat(autoware_adapi_v1_msgs): add operation mode api message (`#17 <https://github.com/youtalk/autoware_adapi_msgs/issues/17>`_)
* feat(autoware_adapi_v1_msgs): add operation mode api message
* feat: use minimum size type
* feat(autoware_adapi_v1_msgs): add velocity factor message (`#15 <https://github.com/youtalk/autoware_adapi_msgs/issues/15>`_)
* feat(autoware_adapi_v1_msgs): add error code to route set service (`#14 <https://github.com/youtalk/autoware_adapi_msgs/issues/14>`_)
* feat(autoware_adapi_v1_msgs): add motion api message (`#11 <https://github.com/youtalk/autoware_adapi_msgs/issues/11>`_)
feat(autoware_adapi_v1_msgs): add motion messages
* feat(autoware_adapi_v1_msgs): add planning api message (`#9 <https://github.com/youtalk/autoware_adapi_msgs/issues/9>`_)
* add planning api msgs
* add cmake
* fix naming
* feat(autoware_adapi_v1_msgs): add v1 messages (`#3 <https://github.com/youtalk/autoware_adapi_msgs/issues/3>`_)
Co-authored-by: Ryohsuke Mitsudome <[email protected]>
* Contributors: Ahmed Ebrahim, Akihiro Sakurai, Kah Hooi Tan, Kosuke Takeuchi, Makoto Kurihara, Takagi, Isamu, Yutaka Kondo, Yutaka Shimizu
2 changes: 1 addition & 1 deletion autoware_adapi_v1_msgs/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<?xml-model href="http://download.ros.org/schema/package_format3.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
<package format="3">
<name>autoware_adapi_v1_msgs</name>
<version>1.2.0</version>
<version>1.3.0</version>
<description>The Autoware AD API interfaces</description>
<maintainer email="[email protected]">Takagi, Isamu</maintainer>
<maintainer email="[email protected]">Yutaka Kondo</maintainer>
Expand Down
1 change: 0 additions & 1 deletion autoware_adapi_v1_msgs/vehicle/msg/VehicleStatus.msg
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,3 @@ autoware_adapi_v1_msgs/Gear gear
autoware_adapi_v1_msgs/TurnIndicators turn_indicators
autoware_adapi_v1_msgs/HazardLights hazard_lights
float64 steering_tire_angle
float32 energy_percentage # Battery percentage or fuel percentage, it will depends on the vehicle.
Loading

0 comments on commit 833bbf3

Please sign in to comment.