-
Notifications
You must be signed in to change notification settings - Fork 2
cpp camera merge to main #29
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
base: main
Are you sure you want to change the base?
Changes from all commits
5c4341e
7a2cfa5
0bd2a9f
5a1f961
d655f4f
68a2c26
9a2c899
e41a02f
0779e1d
9efad18
58c1a45
e923ac6
f78f1b3
4d81c12
3751b45
f34cfff
0ac1bf3
a188a24
d3075cb
9881363
7365581
4c2ebf0
e6f105b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,8 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| source /opt/ros/humble/setup.bash | ||
| colcon build --symlink-install | ||
| [ -f /ws/install/setup.bash ] && source /ws/install/setup.bash | ||
| [ -f /ws/.venv/bin/activate ] && source /ws/.venv/bin/activate | ||
|
|
||
| colcon build | ||
| pkill -f master_api || true | ||
| sleep 0.5 | ||
| ros2 launch autonomous_kart bringup_sim.launch.py |
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| # Camera config settings | ||
| camera_node: | ||
| ros__parameters: | ||
| fps: 60.0 # float | ||
| fps: 0 # 0 == use system frequency |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,26 +1,26 @@ | ||
| from glob import glob | ||
| from setuptools import setup | ||
|
|
||
| package_name = 'autonomous_kart' | ||
| package_name = "autonomous_kart" | ||
|
|
||
| setup( | ||
| name=package_name, | ||
| version="0.0.1", | ||
| packages=[ | ||
| 'autonomous_kart', | ||
| 'autonomous_kart.nodes', | ||
| 'autonomous_kart.nodes.motor', | ||
| 'autonomous_kart.nodes.steering', | ||
| 'autonomous_kart.nodes.camera', | ||
| 'autonomous_kart.nodes.gps', | ||
| 'autonomous_kart.nodes.pathfinder', | ||
| 'autonomous_kart.nodes.opencv_pathfinder', | ||
| 'autonomous_kart.nodes.e_comms', | ||
| 'autonomous_kart.nodes.3dgs_localization', | ||
| 'autonomous_kart.nodes.metrics', | ||
| 'autonomous_kart.nodes.master', | ||
| 'autonomous_kart.nodes.localization', | ||
| 'autonomous_kart.nodes.pathfinder.strategies', | ||
| "autonomous_kart", | ||
| "autonomous_kart.nodes", | ||
| "autonomous_kart.nodes.motor", | ||
| "autonomous_kart.nodes.steering", | ||
| "autonomous_kart.nodes.camera", | ||
| "autonomous_kart.nodes.gps", | ||
| "autonomous_kart.nodes.pathfinder", | ||
| "autonomous_kart.nodes.opencv_pathfinder", | ||
| "autonomous_kart.nodes.e_comms", | ||
| "autonomous_kart.nodes.3dgs_localization", | ||
| "autonomous_kart.nodes.metrics", | ||
| "autonomous_kart.nodes.master", | ||
| "autonomous_kart.nodes.localization", | ||
| "autonomous_kart.nodes.pathfinder.strategies", | ||
| ], | ||
| data_files=[ | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please revert these changes, its irrelevant to the PR.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. While I agree that it is irrelevant to the pr, I disagree that the formatting errors should remain
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We don't have a standard linter for this project, so if the formatting issue is cosmetic and still works, we shouldn't make it. Two competing linters would create extra clutter/merge conflicts. If you are making this change, it should then be for the entire codebase. |
||
| ("share/ament_index/resource_index/packages", ["resource/" + package_name]), | ||
|
|
@@ -30,25 +30,24 @@ | |
| ], | ||
| install_requires=["setuptools"], | ||
| zip_safe=True, | ||
| maintainer='Purdue EVC', | ||
| maintainer_email='shay.manor@gmail.com', | ||
| license='Apache-2.0', | ||
| maintainer="Purdue EVC", | ||
| maintainer_email="shay.manor@gmail.com", | ||
| license="Apache-2.0", | ||
| description="Package containing all nodes for driving in different states.", | ||
| tests_require=["pytest"], | ||
| entry_points={ | ||
| 'console_scripts': [ | ||
| 'motor_node = autonomous_kart.nodes.motor.motor_node:main', | ||
| 'pathfinder_node = autonomous_kart.nodes.pathfinder.pathfinder_node:main', | ||
| 'manual_pathfinder_api = autonomous_kart.nodes.manual.pathfinder_api:main', | ||
| 'gps_node = autonomous_kart.nodes.gps.gps_node:main', | ||
| '3dgs_localization_node = autonomous_kart.nodes.3dgs_localization.3dgs_localization_node:main', | ||
| 'opencv_pathfinder_node = autonomous_kart.nodes.opencv_pathfinder.opencv_pathfinder_node:main', | ||
| 'e_comms_node = autonomous_kart.nodes.e_comms.e_comms_node:main', | ||
| 'steering_node = autonomous_kart.nodes.steering.steering_node:main', | ||
| 'camera_node = autonomous_kart.nodes.camera.camera_node:main', | ||
| 'metrics_node = autonomous_kart.nodes.metrics.metrics_node:main', | ||
| 'master_api = autonomous_kart.nodes.master.master_api:main', | ||
| 'localization_node = autonomous_kart.nodes.localization.localization_node:main', | ||
| "console_scripts": [ | ||
| "motor_node = autonomous_kart.nodes.motor.motor_node:main", | ||
| "pathfinder_node = autonomous_kart.nodes.pathfinder.pathfinder_node:main", | ||
| "manual_pathfinder_api = autonomous_kart.nodes.manual.pathfinder_api:main", | ||
| "gps_node = autonomous_kart.nodes.gps.gps_node:main", | ||
| "3dgs_localization_node = autonomous_kart.nodes.3dgs_localization.3dgs_localization_node:main", | ||
| "opencv_pathfinder_node = autonomous_kart.nodes.opencv_pathfinder.opencv_pathfinder_node:main", | ||
| "e_comms_node = autonomous_kart.nodes.e_comms.e_comms_node:main", | ||
| "steering_node = autonomous_kart.nodes.steering.steering_node:main", | ||
| "metrics_node = autonomous_kart.nodes.metrics.metrics_node:main", | ||
| "master_api = autonomous_kart.nodes.master.master_api:main", | ||
| "localization_node = autonomous_kart.nodes.localization.localization_node:main", | ||
| ], | ||
| }, | ||
| ) | ||
| ) | ||
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 delete