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.

Sign up for GitHub

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

Jump to bottom

feat: (system_monitor) adding a node for CMOS battery monitoring #1841

Closed
wants to merge 179 commits into from
Closed

feat: (system_monitor) adding a node for CMOS battery monitoring #1841

wants to merge 179 commits into from

Conversation

nobuotakamasa
Copy link
Contributor

@nobuotakamasa nobuotakamasa commented Sep 12, 2022 •
edited
Loading

Description

adding "system_monitor/ecu_monitor" node that monitors for CMOS battery health.
This PR is related to the following issue.
#1452

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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

kenji-miyake and others added 30 commits January 24, 2022 18:00
@kenji-miyake
Merge pull request #1 from autowarefoundation/tier4/proposal
23f96c0 
chore: sync upstream
@kenji-miyake
Merge pull request #3 from autowarefoundation/tier4/proposal
a0fe549 
chore: sync upstream
@kenji-miyake
ci: add sync-upstream.yaml (#4)
33549de 
Signed-off-by: Kenji Miyake <[email protected]>
@tier4-autoware-public-bot
Merge pull request #5 from tier4/sync-upstream
355609b 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #11 from tier4/sync-upstream
b6beeec 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #13 from tier4/sync-upstream
f371839 
chore: sync upstream
@kenji-miyake
ci(sync-upstream): update settings (#19)
acf26ea 
Signed-off-by: Kenji Miyake <[email protected]>
@tier4-autoware-public-bot
Merge pull request #17 from tier4/sync-upstream
f92e938 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #20 from tier4/sync-upstream
26c6541 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #27 from tier4/sync-upstream
8082bea 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #29 from tier4/sync-upstream
d065ddd 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #30 from tier4/sync-upstream
4c1d0cb 
chore: sync upstream
@awf-autoware-bot @kenji-miyake
@pre-commit-ci @h-ohta
chore: sync files (#629)
4a3e529 
* chore: sync files

Signed-off-by: GitHub <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: kenji-miyake <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@awf-autoware-bot @kenji-miyake @h-ohta
chore: sync files (#637)
3dea07d 
Signed-off-by: GitHub <[email protected]>

Co-authored-by: kenji-miyake <[email protected]>
@h-ohta
fix(dummy_diag_publisher): use anon to make unique node name instead …
f037c18 
…of diag name (#639)
@tier4-autoware-public-bot
Merge pull request #32 from tier4/sync-upstream
0c2022a 
chore: sync upstream
@yn-mrse
Merge branch 'tier4/main' into sync-upstream
a4e7e2f
@tier4-autoware-public-bot
Merge pull request #33 from tier4/sync-upstream
b9fb070 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #35 from tier4/sync-upstream
840d3e9 
fix(obstacle_stop_planner): use utils function to caluculate nearest index from ego (#651)
@awf-autoware-bot @kenji-miyake @h-ohta
chore: sync files (#648)
3a37242 
* chore: sync files

Signed-off-by: GitHub <[email protected]>

* Revert "chore: sync files"

This reverts commit b24f530.

Signed-off-by: Kenji Miyake <[email protected]>

* sync codecov.yaml

Signed-off-by: Kenji Miyake <[email protected]>

Co-authored-by: kenji-miyake <[email protected]>
Co-authored-by: Kenji Miyake <[email protected]>
@h-ohta @pre-commit-ci
fix(autoware_state_panel): fix message type for /api/autoware/get/eng…
a827200 
…age (#666)

* fix(autoware_state_panel): fix message type for /api/autoware/get/engage

Signed-off-by: h-ohta <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@tier4-autoware-public-bot
Merge pull request #36 from tier4/sync-upstream
cc39553 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #37 from tier4/sync-upstream
75b9e2d 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #39 from tier4/sync-upstream
8278cce 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #41 from tier4/sync-upstream
e0ad67a 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #42 from tier4/sync-upstream
6174d32 
chore: sync upstream
@tier4-autoware-public-bot
Merge pull request #43 from tier4/sync-upstream
65e6d7b 
chore: sync upstream
@taikitanaka3 @0x126
fix(behavior_velocity): avoid insert same point on trajectory utils (#…
9f5837f 
…834)

Signed-off-by: tanaka3 <[email protected]>
@kenji-miyake @0x126
refactor(behavior_velocity_planner): simplify CMakeLists.txt (#855)
69fc674 
Signed-off-by: Kenji Miyake <[email protected]>
@Sharrrrk @pre-commit-ci
@wep21 @0x126
fix(system_monitor): fix build error on tegra platform (#869)
063a95d 
* fix(system_monitor): fix build error on tegra platform

Signed-off-by: Shark Liu <[email protected]>

* ci(pre-commit): autofix

* Update system/system_monitor/src/gpu_monitor/tegra_gpu_monitor.cpp

Co-authored-by: Shark Liu <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Daisuke Nishimatsu <[email protected]>
@satoshi-ota
Copy link
Contributor

@nobuotakamasa This PR seems to contain unnecessary diff, right ? If that is so, could you clean up your feature branch by using rebase or merge, etc,,, ?

@nobuotakamasa
Copy link
Contributor Author

Too many conflicts. Therefore, I remade the following PR.
#1989

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ito-san ito-san ito-san requested changes

@kenji-miyake kenji-miyake Awaiting requested review from kenji-miyake

@YamatoAndo YamatoAndo Awaiting requested review from YamatoAndo

@taikitanaka3 taikitanaka3 Awaiting requested review from taikitanaka3

@tkimura4 tkimura4 Awaiting requested review from tkimura4

@shmpwk shmpwk Awaiting requested review from shmpwk

@TakaHoribe TakaHoribe Awaiting requested review from TakaHoribe

@amc-nu amc-nu Awaiting requested review from amc-nu

@yukkysaito yukkysaito Awaiting requested review from yukkysaito

@satoshi-ota satoshi-ota Awaiting requested review from satoshi-ota

@rej55 rej55 Awaiting requested review from rej55

@soblin soblin Awaiting requested review from soblin

@kyoichi-sugahara kyoichi-sugahara Awaiting requested review from kyoichi-sugahara

@kosuke55 kosuke55 Awaiting requested review from kosuke55

@purewater0901 purewater0901 Awaiting requested review from purewater0901

@TomohitoAndo TomohitoAndo Awaiting requested review from TomohitoAndo

@zulfaqar-azmi-t4 zulfaqar-azmi-t4 Awaiting requested review from zulfaqar-azmi-t4

@takayuki5168 takayuki5168 Awaiting requested review from takayuki5168

@wep21 wep21 Awaiting requested review from wep21

@yukke42 yukke42 Awaiting requested review from yukke42

@s-azumi s-azumi Awaiting requested review from s-azumi

@mitsudome-r mitsudome-r Awaiting requested review from mitsudome-r

@h-ohta h-ohta Awaiting requested review from h-ohta

Assignees

@nobuotakamasa nobuotakamasa

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.