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: resource check #1487

Closed
wants to merge 17 commits into from
Closed

feat: resource check #1487

wants to merge 17 commits into from

Conversation

asa-naki
Copy link

@asa-naki asa-naki commented Aug 27, 2024

Description

beta/v0.3.17(v4.1.0)~beta/v0.3.18(v4.2.0)に取り込まれた機能のうち、可用性を損失してしまう機能変更が含まれていた。

本PRでは、
beta/v0.3.16(v4.0系列)から
このPRはbeta/v0.3.17(v4.1.0),beta/v0.3.18(v4.2.0)で取り込まれた機能のうち、可用性損失が考えられる機能を除外し、
一通り取り込みの再適応を実施する。

取り込み対象となる機能は下記の通りである。

上記機能をcherry-pickを実施した。

Related links

How was this PR tested?

妥当性確認

全体構成の妥当性

取り込み対象となる機能一覧数と同一であることを確認済みです。

それぞれの各論の検証

既存機能をcherry-pickして各機能が動作することから、検証するのを不要と判断しました。

ECUのリソース評価検証

で各機能変更に対して、ECUのリソース負荷状況を計測し、可用性を損失するようなリソース負荷が大きい変更でないことを確認済みです。

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

asa-naki and others added 16 commits August 14, 2024 15:41
* feat(rviz_plugin): console meter is too large on the Rviz with FHD display, isn't it? (#587)

* feat(tier4_planning/vehicle_plugin): make plugins size scalable

Signed-off-by: Takamasa Horibe <[email protected]>

* remove space

Signed-off-by: Takamasa Horibe <[email protected]>

* scaling

Signed-off-by: Takamasa Horibe <[email protected]>

* change diag message

Signed-off-by: asa-naki <[email protected]>

* fix module name

Signed-off-by: asa-naki <[email protected]>

* add manual module and ignoring modules

Signed-off-by: asa-naki <[email protected]>

* Revert "feat(rviz_plugin): console meter is too large on the Rviz with FHD display, isn't it? (#587)"

This reverts commit f96169c.

* Revert "change diag message"

This reverts commit dff01ce.

* ci(pre-commit): autofix

* fix spell check

Signed-off-by: asa-naki <[email protected]>

* Revert "fix spell check"

This reverts commit 208aa1e.

* Revert "fix module name"

This reverts commit cec7653.

* revert ignore module

Signed-off-by: asa-naki <[email protected]>

* current_mode check update

Signed-off-by: asa-naki <[email protected]>

* ci(pre-commit): autofix

* delete margin

Signed-off-by: asa-naki <[email protected]>

---------

Signed-off-by: Takamasa Horibe <[email protected]>
Signed-off-by: asa-naki <[email protected]>
Co-authored-by: Takamasa Horibe <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* fix error_sharp_angle

Signed-off-by: Shigekazu Fukuta <[email protected]>

* update readme

Signed-off-by: Shigekazu Fukuta <[email protected]>

---------

Signed-off-by: Shigekazu Fukuta <[email protected]>
* fix(behavior_path): delete duplicated

* add error handling

* fix: when path size is 1
* fix(detection_area): search collision index only in lanelet

* ci(pre-commit): autofix

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* fix(detection_area): fix overline function

* ci(pre-commit): autofix

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
)

* fix(route_handler): fix threshold for removing overlapping points

* fix
…utowarefoundation#4634) (#880)

fix(ntp_monitor): move chronyc command execution to a timer (autowarefoundation#4634)

* fix(ntp_monitor): move chronyc command execution to a timer



* add newly added parameter timeout to config

---------

Signed-off-by: ito-san <[email protected]>
Co-authored-by: ito-san <[email protected]>
… (backport autowarefoundation#4187) (#652)

feat(elevation_map_loader): add error handling for std::runtime_error (autowarefoundation#4187)

* feat(elevation_map_loader): Add error handling for std::runtime_error



* feat(elevation_map_loader): add error message output



---------

Signed-off-by: Shin-kyoto <[email protected]>
Co-authored-by: Shintaro Tomie <[email protected]>
…efoundation#4553) (#768)

fix(system_monitor): extend command line to display (autowarefoundation#4553)

Signed-off-by: ito-san <[email protected]>
Co-authored-by: ito-san <[email protected]>
…r (backport autowarefoundation#4654) (#769)

fix(system_monitor): high-memory process are not provided in MEM order (autowarefoundation#4654)

* fix(process_monitor): high-memory process are not being provided in %MEM order



* changed option from 'g' to 'n'



---------

Signed-off-by: ito-san <[email protected]>
Co-authored-by: ito-san <[email protected]>
…arefoundation#5191, autowarefoundation#5430) (#995)

* perf(system_monitor): fix program command line reading (autowarefoundation#5191)

* Fix program command line reading

Signed-off-by: Owen-Liuyuxuan <[email protected]>

* style(pre-commit): autofix

* fix spelling commandline->command_line

Signed-off-by: Owen-Liuyuxuan <[email protected]>

---------

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

* fix(system_monitor): output command line (autowarefoundation#5430)

* fix(system_monitor): output command line

Signed-off-by: takeshi.iwanari <[email protected]>

* style(pre-commit): autofix

---------

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

---------

Signed-off-by: Owen-Liuyuxuan <[email protected]>
Signed-off-by: takeshi.iwanari <[email protected]>
Co-authored-by: Yuxuan Liu <[email protected]>
Co-authored-by: Owen-Liuyuxuan <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: takeshi-iwanari <[email protected]>
Co-authored-by: Akihisa Nagata <[email protected]>
…tion#4729) (#856)

* feat(imu_corrector): add gyro_bias_validator (backport autowarefoundation#4729)
* feat(imu_corrector): add gyro_bias_validator
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* update
Signed-off-by: kminoda <[email protected]>
* revert launch
Signed-off-by: kminoda <[email protected]>
* updat
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* add debug publisher
Signed-off-by: kminoda <[email protected]>
* minor fix
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* style(pre-commit): autofix
* add gtest
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* updat e readme
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* add diagnostics
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* update
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* validator -> estimator
Signed-off-by: kminoda <[email protected]>
* fix build
Signed-off-by: kminoda <[email protected]>
* update default parameter
Signed-off-by: kminoda <[email protected]>
* update comment
Signed-off-by: kminoda <[email protected]>
* update readme
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* updated
Signed-off-by: kminoda <[email protected]>
* minor update in readme
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* fix pre-commit
Signed-off-by: kminoda <[email protected]>
* update readme
Signed-off-by: kminoda <[email protected]>
* style(pre-commit): autofix
* Fix NG -> WARN
Signed-off-by: kminoda <[email protected]>
---------
Signed-off-by: kminoda <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Signed-off-by: asa-naki <[email protected]>

* build(imu_corrector): add missing diagnostic_updater dependency (autowarefoundation#4980)

Signed-off-by: Esteve Fernandez <[email protected]>

* add gyro_bias estimation in diag ( autowarefoundation#5054)

Signed-off-by: asa-naki <[email protected]>

* ci(pre-commit): autofix

---------

Signed-off-by: asa-naki <[email protected]>
Signed-off-by: Esteve Fernandez <[email protected]>
Co-authored-by: kminoda <[email protected]>
Co-authored-by: Esteve Fernandez <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
add gyro_bias log

Signed-off-by: asa-naki <[email protected]>
…#888)

* add ignore_module

Signed-off-by: asa-naki <[email protected]>

* add description

Signed-off-by: asa-naki <[email protected]>

* ci(pre-commit): autofix

* change name ignore_until_waiting_for_route

Signed-off-by: asa-naki <[email protected]>

* update description

Signed-off-by: asa-naki <[email protected]>

* rename function name and delete planning state

Signed-off-by: asa-naki <[email protected]>

* update description

Signed-off-by: asa-naki <[email protected]>

* Update

---------

Signed-off-by: asa-naki <[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]>
…ng state (#972)

* add ignore hartbeat timeout in initializing state

Signed-off-by: asa-naki <[email protected]>

* fix typo

* Update comment

* ci(pre-commit): autofix

* fix typo

* ci(pre-commit): autofix

* update comment

* ci(pre-commit): autofix

---------

Signed-off-by: asa-naki <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@asa-naki asa-naki changed the title feat/resource check feat: resource check Aug 27, 2024
* update pre-commit-config

* ci(pre-commit): autofix

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Copy link

sonarcloud bot commented Aug 27, 2024

@asa-naki asa-naki requested a review from yn-mrse August 27, 2024 05:14
@asa-naki asa-naki closed this Aug 28, 2024
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.

3 participants