Skip to content

Commit

Permalink
Tools: Hotfix missing 3 and linters
Browse files Browse the repository at this point in the history
* CI was disabled, we missed these

Signed-off-by: Ryan Friedman <[email protected]>
  • Loading branch information
Ryanf55 authored and peterbarker committed Sep 11, 2024
1 parent 2b903d2 commit 236b3e5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
3 changes: 1 addition & 2 deletions Tools/ros2/ardupilot_dds_tests/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,7 @@
<test_depend>python3-pytest</test_depend>
<test_depend>rclpy</test_depend>
<test_depend>sensor_msgs</test_depend>
<test_depend>python-scipy</test_depend>

<test_depend>python3-scipy</test_depend>

<export>
<build_type>ament_python</build_type>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ def ros_quat_to_heading(quat):


def validate_position_cmac(position):
"""Returns true if the vehicle is at CMAC"""
"""Return true if the vehicle is at CMAC."""
LAT_LON_TOL = 1e-5
return (
math.isclose(position.latitude, CMAC_LAT, abs_tol=LAT_LON_TOL)
Expand All @@ -59,12 +59,12 @@ def validate_position_cmac(position):


def validate_heading_cmac(orientation):
"""Returns true if the vehicle is facing the right way for CMAC"""
"""Return true if the vehicle is facing the right way for CMAC."""
return math.isclose(ros_quat_to_heading(orientation), CMAC_HEADING)


class GeoPoseListener(rclpy.node.Node):
"""Subscribe to GeoPoseStamped messages"""
"""Subscribe to GeoPoseStamped messages."""

def __init__(self):
"""Initialise the node."""
Expand Down

0 comments on commit 236b3e5

Please sign in to comment.