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

Checking for models with dispensable in names to not be considered an obstacle by slotcar #74

Merged
merged 3 commits into from
Jun 27, 2022

Conversation

aaronchongth
Copy link
Member

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

New feature implementation

Implemented feature

Slotcar will not consider models with names that contain dispensable as obstacles. This will allow a robot to carry a model like MiRCart, which has an overlapping pose with the robot.

Spawning this robot using open-rmf/rmf_traffic_editor#436, will allow that to happen.

… obstacle by slotcar

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

codecov bot commented Jun 23, 2022

Codecov Report

Merging #74 (96254f1) into main (23717f1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##            main     #74      +/-   ##
========================================
  Coverage   0.00%   0.00%              
========================================
  Files        348     144     -204     
  Lines      33231   13400   -19831     
========================================
+ Misses     33231   13400   -19831     
Flag Coverage Δ
tests 0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...uilding_sim_gazebo_plugins/src/toggle_charging.cpp
...lation/rmf_robot_sim_common/src/slotcar_common.cpp
...on_plugins/src/toggle_charging/toggle_charging.cpp
...obot_sim_ignition_plugins/src/TeleportIngestor.cpp
...ion/rmf_robot_sim_ignition_plugins/src/slotcar.cpp
...on/include/rmf_robot_sim_common/slotcar_common.hpp
...f_simulation/rmf_building_sim_common/src/utils.cpp
...uilding_sim_gazebo_plugins/src/crowd_simulator.hpp
...tion/rmf_robot_sim_common/src/dispenser_common.cpp
...ion/rmf_building_sim_ignition_plugins/src/door.cpp
... and 466 more

Copy link
Member

@luca-della-vedova luca-della-vedova left a comment

Choose a reason for hiding this comment

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

LGTM! I noticed that the condition we use to initialize infrastructure is not 100% bullet proof (if a map had no obstacle_exclusion we would re-initialize at every iteration). That is an obscure enough case that I guess we can still accept

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

@luca-della-vedova Not really, after initialization there will be at least be itself in the set. I just added in a comment to explain that. It dismissed your review though, if you would be so kind 🙇

Copy link
Member

@luca-della-vedova luca-della-vedova left a comment

Choose a reason for hiding this comment

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

ah of course 🤦‍♂️ thanks!

@aaronchongth aaronchongth merged commit bcd186d into main Jun 27, 2022
@aaronchongth aaronchongth deleted the feature/dispensable_models_not_obstacles branch June 27, 2022 08:37
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.

None yet

2 participants