-
Notifications
You must be signed in to change notification settings - Fork 6
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
Zero shot localization, multi-agent support, convoy #153
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…witt/autonomy-research-testbed into feature/general_autonomy
…t/autonomy-research-testbed into feature/general_autonomy
…t/autonomy-research-testbed into feature/general_autonomy
harryzhang1018
approved these changes
Sep 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added a lot of stuff, not expecting to merge within the next week or so until I do a whole lot of cleaning up (lots of random changes that need to be reverted) and testing on the live ARTs. This may also be better for a branch that's not main. Would also like some opinions on the design decision made. Just a rundown of new features I'm proposing
Multi-agent structuring
robot_ns
parameter (will probably make a util function for this, the PythonExpression's are an eyesore)- Description package
-Multi-agent localization to a global map frame:
map -> robot_ns/odom -> robot_ns/base_link
Localization updates
robot_localization
&navsat_transform_node
with a custom set_datum client to have a shared map frame between ART instanceschrono_imu_filter
package (changing this tochrono_sensor_filter
soon) which acts as an interface between Chrono & R_L. It republishes the Chrono topics as~/gps/fix
,~/imu/data
. Magnetometer calculations are handled and directly thrown into the ~/imu/data message (copied from ground truth heading calculations). This should behave exactly the same as the sensor suite on Oak.Convoy demo
Vehicle configs
art_sensing_launch/config/${VEH_CONFIG}.yml
, ${VEH_CONFIG} is stored in atk.envvehicle
packageDrivers support
robot_ns
namespacingFastDDS Discovery Server & Super Client
ROS_DISCOVERY_SERVER
from atk.envdiscovery
service (no interactive shell is fine)introspection.sh
ROS_DISCOVERY_SERVER=127.0.0.1:11811
, and all other vehicles useROS_DISCOVERY_SERVER=<lead vehicle wlan0>:11811
iftop
with it on and off to profile congestionHardware in the Loop
Right now to launch from
art.launch.py
you just have to use therobot_ns
andleader_ns
parameters (i.e artcar_1, artcar_2) The lead car should useleader_ns:=none
. The leader is the car directly in front of the current car, not the first car in the convoy.Other planned/WIP features:
PythonExpression
substitution support when appending namespaces to topics throughLaunchConfiguration
, also add utility functions for loading vehicle configs