Skip to content
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

Feat/320 adding opponent cars #322

Merged
merged 30 commits into from
Feb 13, 2021
Merged

Conversation

NicoBartholomai
Copy link
Contributor

Fixes issue #320
Adds multiple cars to the track when launching evgp_with_opponent.launch.
Added two new spawning nodes for two extra cars, as well as created three new csv files that determine their path.
Also created 5 new opponent cars in rr_description/urdf/opponents that each have their own color, as well as unique collision bit-masks so that the opponent cars cannot run into each other and lag out.
Adds two new parameters to util/rosbag_to_trajectory_csv.py which adds the functionality to give the kart a starting position other than 0, 0 when creating a path csv from a bag file, since ros does not track what the initial gazebo coordinates are.
Tested by running 'roslaunch rr_gazebo evgp_with_opponent.launch' immediately followed by "roslaunch rr_evgp evgp_planning_demo.launch" and then saw the three cars spawned and moved around the track.

Video of the test run can be found here

@Daniel-Martin576
Copy link
Contributor

Can you update this branch so that the file changed is only the ones you actually touched? Just merge master and re-push pretty much

Copy link
Contributor

@Daniel-Martin576 Daniel-Martin576 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Nice Job! 👍 💯

@Daniel-Martin576 Daniel-Martin576 merged commit 73ff7ec into master Feb 13, 2021
@Daniel-Martin576 Daniel-Martin576 deleted the feat/320-adding-opponent-cars branch February 13, 2021 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants