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

Arjo/experimental/multidrone drag #138

Merged

Conversation

aaronchongth
Copy link
Collaborator

@aaronchongth aaronchongth commented Jun 8, 2022

This PR continues @arjo129's work for lifting and dragging objects using multiple UAVs, and is based on #108.

Few things that this PR does:

  • adds a new world for testing dragging overweight objects, mbzirc_ign/worlds/test/multi_drone_drag.sdf. This adds the Large * Handles models mentioned in Update large objects and vessels geometry #130 into the world
  • modifies the original examples/multi_uav_demo.py script to allow users to specify each velocity of the twist command
  • adds a demo launch script, multi_uav_drag_demo.launch.py: spawns 2 UAVs with suction above the Large Crate Handles model at the front of the vessel
  • adds a demo launch script, multi_uav_drag_with_handles_demo.launch.py, spawns 2 UAVs with suction above the Large Dry Box Handles model at the top of the vessel

Instructions on running,

First terminal,

cd ~/mbzirc_ws/src/mbzirc/mbzirc_ign/worlds/test
ign gazebo multi_drone_drag.sdf

Second terminal,

source ~/mbzirc_ws/install/setup.bash
ros2 launch mbzirc_ign multi_uav_drag_demo.launch.py

# or
# ros2 launch mbzirc_ign multi_uav_drag_with_handles_demo.launch.py

Start the simulation,

Third terminal,

source ~/mbzirc_ws/install/setup.bash
ros2 run mbzirc_ign multi_uav_demo.py -z 10 -y -40 -x 0

The UAVs should start dragging Large Crate Handles to the side of the vessel. The process is much slower since the vessel's surface there is curved, it should be much faster if the drones were maneuvered properly too.

drag_demo.mp4

arjo129 and others added 6 commits May 31, 2022 16:52
Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
…s for dragging

Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
@iche033
Copy link
Collaborator

iche033 commented Jun 8, 2022

works for me! I made some style fixes in #141 which should help to make CI green.

The default inertial values may be be the best but seem to be helping the drones drag the objects. See if it's possible to give them them with proper values and that dragging still works.

@iche033
Copy link
Collaborator

iche033 commented Jun 13, 2022

#143 should fix the detach topic issue when there are multiple drones with grippers. I think there are still more detach issues remaining as mentioned in gazebosim/gz-physics#352 (comment)

iche033 and others added 3 commits June 14, 2022 11:02
* create unique gripper model files per uav

Signed-off-by: Ian Chen <ichen@osrfoundation.org>

* style

Signed-off-by: Ian Chen <ichen@osrfoundation.org>

* Checking if gipper model name is a substring instead of direct compare (#145)

Signed-off-by: Aaron Chong <aaronchongth@gmail.com>

Co-authored-by: Aaron Chong <aaronchongth@gmail.com>
@iche033
Copy link
Collaborator

iche033 commented Jun 16, 2022

updated all Large * Handles models on Fuel. Dragging is now working well for me. We still need the detachable joint fix in gazebosim/gz-physics#352 (comment) but that shouldn't block this PR

Signed-off-by: Ian Chen <ichen@osrfoundation.org>
@aaronchongth
Copy link
Collaborator Author

@iche033 Should we be concerned that the tests have timed out?

@iche033
Copy link
Collaborator

iche033 commented Jun 17, 2022

I thought it's flaky but I restarted the test a few times and it's still failing. I'm also seeing the test failure locally. There could be an issue. I'll take a closer look tomorrow.

Signed-off-by: Ian Chen <ichen@osrfoundation.org>
Signed-off-by: Ian Chen <ichen@osrfoundation.org>
@aaronchongth
Copy link
Collaborator Author

Thanks for the fix! I'll merge this so we can move onto the next PR, #108

@aaronchongth aaronchongth merged commit 05f14b9 into arjo/demo/multidrone_lift Jun 20, 2022
@aaronchongth aaronchongth deleted the arjo/experimental/multidrone_drag branch June 20, 2022 02:40
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