-
Notifications
You must be signed in to change notification settings - Fork 672
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
T4 pb 12150 #1039
Closed
Closed
T4 pb 12150 #1039
Changes from all commits
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
23f96c0
Merge pull request #1 from autowarefoundation/tier4/proposal
kenji-miyake a0fe549
Merge pull request #3 from autowarefoundation/tier4/proposal
kenji-miyake 33549de
ci: add sync-upstream.yaml (#4)
kenji-miyake 355609b
Merge pull request #5 from tier4/sync-upstream
tier4-autoware-public-bot[bot] b6beeec
Merge pull request #11 from tier4/sync-upstream
tier4-autoware-public-bot[bot] f371839
Merge pull request #13 from tier4/sync-upstream
tier4-autoware-public-bot[bot] acf26ea
ci(sync-upstream): update settings (#19)
kenji-miyake f92e938
Merge pull request #17 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 26c6541
Merge pull request #20 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 8082bea
Merge pull request #27 from tier4/sync-upstream
tier4-autoware-public-bot[bot] d065ddd
Merge pull request #29 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 4c1d0cb
Merge pull request #30 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 4a3e529
chore: sync files (#629)
awf-autoware-bot[bot] 3dea07d
chore: sync files (#637)
awf-autoware-bot[bot] f037c18
fix(dummy_diag_publisher): use anon to make unique node name instead …
h-ohta 0c2022a
Merge pull request #32 from tier4/sync-upstream
tier4-autoware-public-bot[bot] a4e7e2f
Merge branch 'tier4/main' into sync-upstream
yn-mrse b9fb070
Merge pull request #33 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 840d3e9
Merge pull request #35 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 3a37242
chore: sync files (#648)
awf-autoware-bot[bot] a827200
fix(autoware_state_panel): fix message type for /api/autoware/get/eng…
h-ohta cc39553
Merge pull request #36 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 75b9e2d
Merge pull request #37 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 8278cce
Merge pull request #39 from tier4/sync-upstream
tier4-autoware-public-bot[bot] e0ad67a
Merge pull request #41 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 6174d32
Merge pull request #42 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 65e6d7b
Merge pull request #43 from tier4/sync-upstream
tier4-autoware-public-bot[bot] 9f5837f
fix(behavior_velocity): avoid insert same point on trajectory utils (…
taikitanaka3 69fc674
refactor(behavior_velocity_planner): simplify CMakeLists.txt (#855)
kenji-miyake 063a95d
fix(system_monitor): fix build error on tegra platform (#869)
Sharrrrk 35a1f0a
feat(ad_service_state_monitor): limit odometry buffer size (#514)
takayuki5168 82c6ad8
docs: fix 404 error caused by typo in url (#871)
Shin-kyoto a7118b6
fix(image_projection_based_fusion): set imagebuffersize (#820)
storrrrrrrrm 8809673
chore(avoidance_module): fix spell check (#732)
zulfaqar-azmi-t4 cc43203
feat: isolate gtests in all packages (#693)
maxime-clem 0786fa1
docs(virtual traffic light): add documentation (#245)
taikitanaka3 4f4644f
feat(surround_obstacle_checker): separate surround_obstacle_checker f…
satoshi-ota 1f5ade6
feat: remove deprecated package in prediction launch (#875)
yukkysaito 399c133
fix(surround_obstacle_checker): fix ego footprint polygon (#877)
satoshi-ota eca2944
fix: update nvinfer api (#863)
wep21 3e5e6db
fix(avoidance_module): ignore object instead of creating zero shift (…
zulfaqar-azmi-t4 7b81507
fix(motion_velocity_smoother): curve deceleration not working with a …
TakaHoribe 6668873
test(autoware_testing): fix smoke_test (#479)
KeisukeShima 31147a2
feat(rviz_plugins): add velocity limit to autoware state panel (#879)
taikitanaka3 da41a37
fix(dummy_perception_publisher): publish multiple layers of pointclou…
HiroIshida e2e557d
feat(vehicle_info_util): add max_steer_angle (#740)
takayuki5168 952510b
ci(deploy-docs): remove mdx_unimoji (#883)
shmpwk b2727f0
chore: sync files (#884)
awf-autoware-bot[bot] ae80db9
fix(lidar_centerpoint): fix google drive url to avoid 404 (#889)
h-ohta c396b9d
chore: fix typos (#886)
badai-nguyen ffb0553
feat(state_rviz_plugin): add GateMode and PathChangeApproval Button (…
h-ohta 80f3f1c
feat(map_tf_generator): accelerate the 'viewer' coordinate calculatio…
IshitaTakeshi ef37953
docs(obstacle_stop_planner): update documentation (#880)
satoshi-ota 9470442
docs(tier4_traffic_light_rviz_plugin): update documentation (#905)
satoshi-ota e1e81b3
fix(accel_brake_map_calibrator): rviz panel type (#895)
soblin 55d807e
fix(behavior velocity planner): skipping emplace back stop reason if …
TakumiKozaka-T4 f1d2b9e
feat(behavior_path_planner): weakened noise filtering of drivable are…
takayuki5168 0621852
Merge pull request #44 from tier4/sync-upstream
tier4-autoware-public-bot[bot] db25a76
Merge branch 'tier4/main' into sync-upstream
YoheiMishina fefdd4d
Merge pull request #47 from tier4/sync-upstream
tier4-autoware-public-bot[bot] f97557e
first commit
nobuotakamasa ab73b5a
fix comment
nobuotakamasa cb13c5d
fix diff
nobuotakamasa e911915
fix comment
nobuotakamasa File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
name: sync-upstream | ||
|
||
on: | ||
schedule: | ||
- cron: 0 0 * * * | ||
workflow_dispatch: | ||
|
||
jobs: | ||
sync-upstream: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Generate token | ||
id: generate-token | ||
uses: tibdex/github-app-token@v1 | ||
with: | ||
app_id: ${{ secrets.APP_ID }} | ||
private_key: ${{ secrets.PRIVATE_KEY }} | ||
|
||
- name: Run sync-branches | ||
uses: autowarefoundation/autoware-github-actions/sync-branches@v1 | ||
with: | ||
token: ${{ steps.generate-token.outputs.token }} | ||
base-branch: tier4/main | ||
sync-pr-branch: sync-upstream | ||
sync-target-repository: https://github.com/autowarefoundation/autoware.universe.git | ||
sync-target-branch: main | ||
pr-title: "chore: sync upstream" | ||
auto-merge-method: merge |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
50 changes: 50 additions & 0 deletions
50
system/system_monitor/include/system_monitor/ecu_monitor/adlink_ecu_monitor.hpp
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,50 @@ | ||
// Copyright 2020 Tier IV, Inc. | ||
// | ||
// Licensed under the Apache License, Version 2.0 (the "License"); | ||
// you may not use this file except in compliance with the License. | ||
// You may obtain a copy of the License at | ||
// | ||
// http://www.apache.org/licenses/LICENSE-2.0 | ||
// | ||
// Unless required by applicable law or agreed to in writing, software | ||
// distributed under the License is distributed on an "AS IS" BASIS, | ||
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
// See the License for the specific language governing permissions and | ||
// limitations under the License. | ||
|
||
/** | ||
* @file adlink_ecu_monitor.h | ||
* @brief ECU monitor class | ||
*/ | ||
|
||
#ifndef SYSTEM_MONITOR__ECU_MONITOR__ADLINK_ECU_MONITOR_HPP_ | ||
#define SYSTEM_MONITOR__ECU_MONITOR__ADLINK_ECU_MONITOR_HPP_ | ||
|
||
#include "system_monitor/ecu_monitor/ecu_monitor_base.hpp" | ||
|
||
#include <string> | ||
|
||
class ECUMonitor : public ECUMonitorBase | ||
{ | ||
public: | ||
/** | ||
* @brief constructor | ||
* @param [in] options Options associated with this node. | ||
*/ | ||
explicit ECUMonitor(const rclcpp::NodeOptions & options); | ||
|
||
protected: | ||
/** | ||
* @brief check ECU thermal throttling | ||
* @param [out] stat diagnostic message passed directly to diagnostic publish calls | ||
* @note NOLINT syntax is needed since diagnostic_updater asks for a non-const reference | ||
* to pass diagnostic message updated in this function to diagnostic publish calls. | ||
*/ | ||
void checkVoltage( | ||
diagnostic_updater::DiagnosticStatusWrapper & stat); // NOLINT(runtime/references) | ||
bool sensors_exists_; //!< @brief flag if sensors exists | ||
float voltage_warn_; | ||
float voltage_error_; | ||
}; | ||
|
||
#endif // SYSTEM_MONITOR__ECU_MONITOR__ADLINK_ECU_MONITOR_HPP_ |
53 changes: 53 additions & 0 deletions
53
system/system_monitor/include/system_monitor/ecu_monitor/ecu_monitor_base.hpp
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
// Copyright 2020 Tier IV, Inc. | ||
// | ||
// Licensed under the Apache License, Version 2.0 (the "License"); | ||
// you may not use this file except in compliance with the License. | ||
// You may obtain a copy of the License at | ||
// | ||
// http://www.apache.org/licenses/LICENSE-2.0 | ||
// | ||
// Unless required by applicable law or agreed to in writing, software | ||
// distributed under the License is distributed on an "AS IS" BASIS, | ||
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
// See the License for the specific language governing permissions and | ||
// limitations under the License. | ||
|
||
/** | ||
* @file ecu_monitor_base.h | ||
* @brief ECU monitor base class | ||
*/ | ||
|
||
#ifndef SYSTEM_MONITOR__ECU_MONITOR__ECU_MONITOR_BASE_HPP_ | ||
#define SYSTEM_MONITOR__MONITOR__ECU_MONITOR_BASE_HPP_ | ||
|
||
#include <diagnostic_updater/diagnostic_updater.hpp> | ||
|
||
#include <climits> | ||
#include <map> | ||
#include <string> | ||
#include <vector> | ||
|
||
class ECUMonitorBase : public rclcpp::Node | ||
{ | ||
public: | ||
/** | ||
* @brief Update the diagnostic state. | ||
*/ | ||
void update(); | ||
|
||
protected: | ||
using DiagStatus = diagnostic_msgs::msg::DiagnosticStatus; | ||
|
||
/** | ||
* @brief constructor | ||
* @param [in] node_name Name of the node. | ||
* @param [in] options Options associated with this node. | ||
*/ | ||
ECUMonitorBase(const std::string & node_name, const rclcpp::NodeOptions & options); | ||
|
||
diagnostic_updater::Updater updater_; //!< @brief Updater class which advertises to /diagnostics | ||
|
||
char hostname_[HOST_NAME_MAX + 1]; //!< @brief host name | ||
}; | ||
|
||
#endif // SYSTEM_MONITOR_ECU_MONITOR_ECU_MONITOR_BASE_HPP_ |
39 changes: 39 additions & 0 deletions
39
system/system_monitor/include/system_monitor/ecu_monitor/unknown_ecu_monitor.hpp
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
// Copyright 2020 Tier IV, Inc. | ||
// | ||
// Licensed under the Apache License, Version 2.0 (the "License"); | ||
// you may not use this file except in compliance with the License. | ||
// You may obtain a copy of the License at | ||
// | ||
// http://www.apache.org/licenses/LICENSE-2.0 | ||
// | ||
// Unless required by applicable law or agreed to in writing, software | ||
// distributed under the License is distributed on an "AS IS" BASIS, | ||
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
// See the License for the specific language governing permissions and | ||
// limitations under the License. | ||
|
||
/** | ||
* @file unknown_ecu_monitor.h | ||
* @brief Unknown ECU monitor class | ||
*/ | ||
|
||
#ifndef SYSTEM_MONITOR_ECU_MONITOR__UNKNOWN_ECU_MONITOR_HPP_ | ||
#define SYSTEM_MONITOR__ECU_MONITOR__UNKNOWN_ECU_MONITOR_HPP_ | ||
|
||
#include "system_monitor/ecu_monitor/ecu_monitor_base.hpp" | ||
|
||
#include <string> | ||
|
||
class ECUMonitor : public ECUMonitorBase | ||
{ | ||
public: | ||
/** | ||
* @brief constructor | ||
* @param [in] node_name Name of the node. | ||
* @param [in] options Options associated with this node. | ||
*/ | ||
explicit ECUMonitor(const rclcpp::NodeOptions & options); | ||
|
||
}; | ||
|
||
#endif // SYSTEM_MONITOR__ECU_MONITOR__UNKNOWN_ECU_MONITOR_HPP_ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
105 changes: 105 additions & 0 deletions
105
system/system_monitor/src/ecu_monitor/adlink_ecu_monitor.cpp
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,105 @@ | ||
// Copyright 2020 Autoware Foundation | ||
// | ||
// Licensed under the Apache License, Version 2.0 (the "License"); | ||
// you may not use this file except in compliance with the License. | ||
// You may obtain a copy of the License at | ||
// | ||
// http://www.apache.org/licenses/LICENSE-2.0 | ||
// | ||
// Unless required by applicable law or agreed to in writing, software | ||
// distributed under the License is distributed on an "AS IS" BASIS, | ||
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
// See the License for the specific language governing permissions and | ||
// limitations under the License. | ||
|
||
/** | ||
* @file _cpu_monitor.cpp | ||
* @brief CPU monitor class | ||
*/ | ||
|
||
#include "system_monitor/ecu_monitor/adlink_ecu_monitor.hpp" | ||
|
||
#include "system_monitor/system_monitor_utility.hpp" | ||
|
||
#include <msr_reader/msr_reader.hpp> | ||
|
||
#include <boost/algorithm/string.hpp> | ||
#include <boost/archive/text_iarchive.hpp> | ||
#include <boost/filesystem.hpp> | ||
#include <boost/process.hpp> | ||
|
||
#include <fmt/format.h> | ||
#include <netinet/in.h> | ||
#include <sys/socket.h> | ||
|
||
#include <algorithm> | ||
#include <regex> | ||
#include <string> | ||
#include <vector> | ||
#include <regex> | ||
|
||
namespace bp = boost::process; | ||
|
||
ECUMonitor::ECUMonitor(const rclcpp::NodeOptions & options) : ECUMonitorBase("ecu_monitor", options) | ||
{ | ||
voltage_warn_ = declare_parameter<float>("low cmos battery warn", 2.9); | ||
voltage_error_ = declare_parameter<float>("low cmos battery error", 2.7); | ||
|
||
updater_.add("ECU CMOS Battery Voltage", this, &ECUMonitor::checkVoltage); | ||
gethostname(hostname_, sizeof(hostname_)); | ||
// Check if command exists | ||
fs::path p = bp::search_path("sensors"); | ||
sensors_exists_ = (p.empty()) ? false : true; | ||
} | ||
|
||
static float getVoltage() { | ||
bp::ipstream is_out; | ||
bp::ipstream is_err; | ||
fs::path p = bp::search_path("sensors"); | ||
bp::child c(p.string(), bp::std_out > is_out, bp::std_err > is_err); | ||
c.wait(); | ||
|
||
if(RCUTILS_UNLIKELY(c.exit_code() != 0)) {//failed to execute sensors | ||
return 0; | ||
} | ||
std::string line; | ||
std::regex re(R"((\d+).(\d+))"); //in7: 3.06 V (min = +0.00 V, max = +4.08 V) | ||
for(int i = 0; i < 200 && std::getline(is_out, line); i++) { | ||
auto voltageStringPos = line.find("in7:"); | ||
if( voltageStringPos != std::string::npos) { | ||
std::smatch match; | ||
std::regex_search(line, match, re); | ||
auto voltageString = match.str(); | ||
return std::stof(voltageString); | ||
} | ||
} | ||
return 0;//failed to read voltage | ||
} | ||
|
||
void ECUMonitor::checkVoltage(diagnostic_updater::DiagnosticStatusWrapper & stat) | ||
{ | ||
// Remember start time to measure elapsed time | ||
const auto t_start = SystemMonitorUtility::startMeasurement(); | ||
|
||
if ( RCUTILS_UNLIKELY(!sensors_exists_) ) { | ||
stat.summary(DiagStatus::ERROR, "sensors error"); | ||
stat.add( | ||
"sensors", "Command 'sensors' not found, but can be installed with: 'sudo apt install lm-sensors' and 'sudo sensors-detect'"); | ||
return; | ||
} | ||
|
||
auto v = getVoltage(); | ||
|
||
stat.add("CMOS battey voltage", fmt::format("{}",v)); | ||
if( RCUTILS_UNLIKELY(v < voltage_warn_) ) { | ||
stat.summary(DiagStatus::WARN, "LOW BATTERY"); | ||
} else { | ||
stat.summary(DiagStatus::OK, "OK"); | ||
} | ||
|
||
// Measure elapsed time since start time and report | ||
SystemMonitorUtility::stopMeasurement(t_start, stat); | ||
} | ||
|
||
#include <rclcpp_components/register_node_macro.hpp> | ||
RCLCPP_COMPONENTS_REGISTER_NODE(ECUMonitor) |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please confirm your diff.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted unnecessary fixes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mm, there is no change. 🥺
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove
sync-upstream.yaml
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, please confirm the pull request guidelines.