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

ci(pre-commit): autoupdate #8949

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ ci:

repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.6.0
rev: v5.0.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.41.0
rev: v0.42.0
hooks:
- id: markdownlint
args: [-c, .markdownlint.yaml, --fix]
Expand Down Expand Up @@ -49,7 +49,7 @@ repos:
- id: shellcheck

- repo: https://github.com/scop/pre-commit-shfmt
rev: v3.9.0-1
rev: v3.10.0-1
hooks:
- id: shfmt
args: [-w, -s, -i=4]
Expand All @@ -60,26 +60,26 @@ repos:
- id: isort

- repo: https://github.com/psf/black
rev: 24.8.0
rev: 24.10.0
hooks:
- id: black
args: [--line-length=100]

- repo: https://github.com/pre-commit/mirrors-clang-format
rev: v18.1.8
rev: v19.1.3
hooks:
- id: clang-format
types_or: [c++, c, cuda]

- repo: https://github.com/cpplint/cpplint
rev: 1.6.1
rev: 2.0.0
hooks:
- id: cpplint
args: [--quiet]
exclude: .cu

- repo: https://github.com/python-jsonschema/check-jsonschema
rev: 0.29.2
rev: 0.29.4
hooks:
- id: check-metaschema
files: ^.+/schema/.*schema\.json$
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -276,7 +276,7 @@ bool CTRVMotionModel::predictStateStep(const double dt, KalmanFilter & ekf) cons
Eigen::MatrixXd X_next_t(DIM, 1); // predicted state
X_next_t(IDX::X) = X_t(IDX::X) + X_t(IDX::VEL) * cos_yaw * dt; // dx = v * cos(yaw)
X_next_t(IDX::Y) = X_t(IDX::Y) + X_t(IDX::VEL) * sin_yaw * dt; // dy = v * sin(yaw)
X_next_t(IDX::YAW) = X_t(IDX::YAW) + (X_t(IDX::WZ)) * dt; // dyaw = omega
X_next_t(IDX::YAW) = X_t(IDX::YAW) + (X_t(IDX::WZ))*dt; // dyaw = omega
X_next_t(IDX::VEL) = X_t(IDX::VEL);
X_next_t(IDX::WZ) = X_t(IDX::WZ);

Expand Down
4 changes: 2 additions & 2 deletions planning/autoware_surround_obstacle_checker/src/node.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -411,8 +411,8 @@ std::optional<geometry_msgs::msg::TransformStamped> SurroundObstacleCheckerNode:

auto SurroundObstacleCheckerNode::isStopRequired(
const bool is_obstacle_found, const bool is_vehicle_stopped, const State & state,
const std::optional<rclcpp::Time> & last_obstacle_found_time,
const double time_threshold) const -> std::pair<bool, std::optional<rclcpp::Time>>
const std::optional<rclcpp::Time> & last_obstacle_found_time, const double time_threshold) const
-> std::pair<bool, std::optional<rclcpp::Time>>
{
if (!is_vehicle_stopped) {
return std::make_pair(false, std::nullopt);
Expand Down
4 changes: 2 additions & 2 deletions planning/autoware_surround_obstacle_checker/src/node.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -89,8 +89,8 @@ class SurroundObstacleCheckerNode : public rclcpp::Node

auto isStopRequired(
const bool is_obstacle_found, const bool is_vehicle_stopped, const State & state,
const std::optional<rclcpp::Time> & last_obstacle_found_time,
const double time_threshold) const -> std::pair<bool, std::optional<rclcpp::Time>>;
const std::optional<rclcpp::Time> & last_obstacle_found_time, const double time_threshold) const
-> std::pair<bool, std::optional<rclcpp::Time>>;

// ros
mutable tf2_ros::Buffer tf_buffer_{get_clock()};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ class SurroundObstacleCheckerNodeTest : public ::testing::Test

auto isStopRequired(
const bool is_obstacle_found, const bool is_vehicle_stopped, const State & state,
const std::optional<rclcpp::Time> & last_obstacle_found_time,
const double time_threshold) const -> std::pair<bool, std::optional<rclcpp::Time>>
const std::optional<rclcpp::Time> & last_obstacle_found_time, const double time_threshold) const
-> std::pair<bool, std::optional<rclcpp::Time>>
{
return node_->isStopRequired(
is_obstacle_found, is_vehicle_stopped, state, last_obstacle_found_time, time_threshold);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@ auto getOffsetPoint(const Eigen::Vector2d & src, const Eigen::Vector2d & dst, co
}

auto findNearestCollisionPoint(
const LineString2d & line1, const LineString2d & line2,
const Point2d & origin) -> std::optional<Point2d>
const LineString2d & line1, const LineString2d & line2, const Point2d & origin)
-> std::optional<Point2d>
{
std::vector<Point2d> collision_points;
bg::intersection(line1, line2, collision_points);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ auto getOffsetPoint(const Eigen::Vector2d & src, const Eigen::Vector2d & dst, co
* @return intersection point. if there is no intersection point, return std::nullopt.
*/
auto findNearestCollisionPoint(
const LineString2d & line1, const LineString2d & line2,
const Point2d & origin) -> std::optional<Point2d>;
const LineString2d & line1, const LineString2d & line2, const Point2d & origin)
-> std::optional<Point2d>;

/**
* @brief find intersection point between path and stop line and return the point.
Expand Down
7 changes: 4 additions & 3 deletions vehicle/autoware_accel_brake_map_calibrator/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -212,10 +212,11 @@ Update by Recursive Least Squares(RLS) method using data close enough to each gr
#### Parameters

Data selection is determined by the following thresholds.
| Name | Default Value |

| Name | Default Value |
| ----------------------- | ------------- |
| velocity_diff_threshold | 0.556 |
| pedal_diff_threshold | 0.03 |
| velocity_diff_threshold | 0.556 |
| pedal_diff_threshold | 0.03 |

#### Update formula

Expand Down
Loading