Skip to content

Modularizing Perception Stack for multiple ROS Packages #7

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.

Sign up for GitHub

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

Open
conjeevaram wants to merge 5 commits into
base: main
Choose a base branch
from

creating modularity within perception module for several packages - c…

74fbba6
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

Modularizing Perception Stack for multiple ROS Packages #7

creating modularity within perception module for several packages - c…
74fbba6
Select commit
Loading
Failed to load commit list.
GitHub Actions / Autopep8 failed May 28, 2025 in 0s

6 errors

Autopep8 found 6 errors

Annotations

Check failure on line 13 in autonomy/samples/python/aggregator/setup.py

See this annotation in the file changed.

@github-actions github-actions / Autopep8

autonomy/samples/python/aggregator/setup.py#L1-L13 

+from setuptools import setup
 import os
-from glob    install_requires=['setuptools'],
-    zip_safe=True,
-    maintainer='eddyzhou, aryanafrouzi',
-    maintainer_email='e23zhou@watonomous.ca, aafrouzi@watonomous.ca',
-    description='Sample aggregator node for data collection',
-    license='Apache2.0',
-    tests_require=['pytest'], glob
-from setuptools import setup
+from glob    install_requires = ['setuptools'],
+zip_safe = True,
+maintainer = 'eddyzhou, aryanafrouzi',
+maintainer_email = 'e23zhou@watonomous.ca, aafrouzi@watonomous.ca',
+description = 'Sample aggregator node for data collection',
+license = 'Apache2.0',
+tests_require = ['pytest'], glob
 
 package_name = 'aggregator'

Check failure on line 33 in src/perception/perception_launch/launch/perception.launch.py

See this annotation in the file changed.

@github-actions github-actions / Autopep8

src/perception/perception_launch/launch/perception.launch.py#L26-L33 

         'config',
         'params.yaml'
     )
-    
+
     # Load pose estimation parameters
     pose_param_file_path = os.path.join(
         get_package_share_directory('pose_estimation'),

Check failure on line 39 in src/perception/depth_estimation/depth_estimation/depth_estimation_core.py

See this annotation in the file changed.

@github-actions github-actions / Autopep8

src/perception/depth_estimation/depth_estimation/depth_estimation_core.py#L33-L39 

     def serialize_data(self):
         return "x:" + str(self.__pos_x) + ";y:" + \
             str(self.__pos_y) + ";z:" + str(self.__pos_z) + ";"
-
-
-

Check failure on line 37 in src/perception/depth_estimation/depth_estimation/depth_estimation_node.py

See this annotation in the file changed.

@github-actions github-actions / Autopep8

src/perception/depth_estimation/depth_estimation/depth_estimation_node.py#L27-L37 

         pass
 
 
-
 def main(args=None):
     rclpy.init(args=args)
     return
 
+
 if __name__ == '__main__':
     main()

Check failure on line 37 in src/perception/pose_estimation/pose_estimation/pose_estimation_node.py

See this annotation in the file changed.

@github-actions github-actions / Autopep8

src/perception/pose_estimation/pose_estimation/pose_estimation_node.py#L27-L37 

         pass
 
 
-
 def main(args=None):
     rclpy.init(args=args)
     return
 
+
 if __name__ == '__main__':
     main()

Check failure on line 39 in src/perception/pose_estimation/pose_estimation/pose_estimation_core.py

See this annotation in the file changed.

@github-actions github-actions / Autopep8

src/perception/pose_estimation/pose_estimation/pose_estimation_core.py#L33-L39 

     def serialize_data(self):
         return "x:" + str(self.__pos_x) + ";y:" + \
             str(self.__pos_y) + ";z:" + str(self.__pos_z) + ";"
-
-
-
View more details on GitHub Actions