Skip to content

Commit

Permalink
ci(pre-commit): autofix
Browse files Browse the repository at this point in the history
  • Loading branch information
pre-commit-ci[bot] committed Oct 21, 2024
1 parent 8178948 commit c2f758e
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
6 changes: 4 additions & 2 deletions nebula_ros/src/hesai/hesai_ros_wrapper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,8 @@ HesaiRosWrapper::HesaiRosWrapper(const rclcpp::NodeOptions & options)
if (use_udp_only) {
RCLCPP_INFO_STREAM(

Check warning on line 46 in nebula_ros/src/hesai/hesai_ros_wrapper.cpp

View check run for this annotation

Codecov / codecov/patch

nebula_ros/src/hesai/hesai_ros_wrapper.cpp#L46

Added line #L46 was not covered by tests
get_logger(),
"UDP-only mode is enabled. Settings checks, synchronization, and diagnostics publishing are disabled.");
"UDP-only mode is enabled. Settings checks, synchronization, and diagnostics publishing are "
"disabled.");
}

if (launch_hw_) {
Expand All @@ -55,7 +56,8 @@ HesaiRosWrapper::HesaiRosWrapper(const rclcpp::NodeOptions & options)
}
}

bool force_load_caibration_from_file = use_udp_only; // Downloading from device requires TCP connection
bool force_load_caibration_from_file =

Check warning on line 59 in nebula_ros/src/hesai/hesai_ros_wrapper.cpp

View check run for this annotation

Codecov / codecov/patch

nebula_ros/src/hesai/hesai_ros_wrapper.cpp#L59

Added line #L59 was not covered by tests
use_udp_only; // Downloading from device requires TCP connection
auto calibration_result =
get_calibration_data(sensor_cfg_ptr_->calibration_path, force_load_caibration_from_file);

Check warning on line 62 in nebula_ros/src/hesai/hesai_ros_wrapper.cpp

View check run for this annotation

Codecov / codecov/patch

nebula_ros/src/hesai/hesai_ros_wrapper.cpp#L62

Added line #L62 was not covered by tests
if (!calibration_result.has_value()) {
Expand Down
3 changes: 1 addition & 2 deletions nebula_ros/src/velodyne/hw_interface_wrapper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@ namespace nebula::ros

VelodyneHwInterfaceWrapper::VelodyneHwInterfaceWrapper(
rclcpp::Node * const parent_node,
std::shared_ptr<const nebula::drivers::VelodyneSensorConfiguration> & config,
bool use_udp_only)
std::shared_ptr<const nebula::drivers::VelodyneSensorConfiguration> & config, bool use_udp_only)

Check warning on line 10 in nebula_ros/src/velodyne/hw_interface_wrapper.cpp

View check run for this annotation

Codecov / codecov/patch

nebula_ros/src/velodyne/hw_interface_wrapper.cpp#L10

Added line #L10 was not covered by tests
: hw_interface_(new nebula::drivers::VelodyneHwInterface()),
logger_(parent_node->get_logger().get_child("HwInterfaceWrapper")),
status_(Status::NOT_INITIALIZED),
Expand Down
3 changes: 2 additions & 1 deletion nebula_ros/src/velodyne/velodyne_ros_wrapper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@ VelodyneRosWrapper::VelodyneRosWrapper(const rclcpp::NodeOptions & options)
if (use_udp_only) {
RCLCPP_INFO_STREAM(

Check warning on line 33 in nebula_ros/src/velodyne/velodyne_ros_wrapper.cpp

View check run for this annotation

Codecov / codecov/patch

nebula_ros/src/velodyne/velodyne_ros_wrapper.cpp#L33

Added line #L33 was not covered by tests
get_logger(),
"UDP-only mode is enabled. Settings checks, synchronization, and diagnostics publishing are disabled.");
"UDP-only mode is enabled. Settings checks, synchronization, and diagnostics publishing are "
"disabled.");
}

if (launch_hw_) {
Expand Down

0 comments on commit c2f758e

Please sign in to comment.