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

feat(aip_x2_launch): add topic state monitor for sensing diags #288

Merged
merged 3 commits into from
Aug 27, 2024

Conversation

kotaro-hihara
Copy link

add topic_state_monitor (radar, imu,gnss).
add imu_monitor to activate sensing diags

related PR https://github.com/tier4/autoware_launch.x2/pull/769

TomohitoAndo and others added 3 commits August 27, 2024 12:03
…and imu (#227)

* feat: add topic state monitor for sensors

Signed-off-by: Tomohito Ando <[email protected]>

* fix comments

Signed-off-by: Tomohito Ando <[email protected]>

* replace launch.xml with launch.py

Signed-off-by: Tomohito Ando <[email protected]>

* ci(pre-commit): autofix

* add copyright

Signed-off-by: Tomohito Ando <[email protected]>

---------

Signed-off-by: Tomohito Ando <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Copy link

@saka1-s saka1-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kotaro-hihara kotaro-hihara merged commit 7aef84c into beta/v0.29.0 Aug 27, 2024
9 of 10 checks passed
@kotaro-hihara kotaro-hihara deleted the feat/topic_state_monitor branch August 27, 2024 08:13
TomohitoAndo added a commit that referenced this pull request Aug 30, 2024
* feat(aip_x2_launch): add topic state monitor for camera, radar, gnss and imu (#227)

* feat: add topic state monitor for sensors

Signed-off-by: Tomohito Ando <[email protected]>

* fix comments

Signed-off-by: Tomohito Ando <[email protected]>

* replace launch.xml with launch.py

Signed-off-by: Tomohito Ando <[email protected]>

* ci(pre-commit): autofix

* add copyright

Signed-off-by: Tomohito Ando <[email protected]>

---------

Signed-off-by: Tomohito Ando <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* remove topic_state_monitor for camera

* feat: add imu_monitor and dummy diag (#212)

---------

Signed-off-by: Tomohito Ando <[email protected]>
Co-authored-by: Tomohito ANDO <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Hiroki OTA <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants