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

[Feature request]: Remove rmf_robot_sim_common package #135

Open
1 task done
luca-della-vedova opened this issue Sep 25, 2024 · 0 comments
Open
1 task done

[Feature request]: Remove rmf_robot_sim_common package #135

luca-della-vedova opened this issue Sep 25, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@luca-della-vedova
Copy link
Member

Before proceeding, is there an existing issue or discussion for this?

Description

The rmf_robot_sim_common package is no longer necessary since we don't need to support gazebo classic anymore, on the contrary it makes the code for plugins unnecessarily complicated (i.e. having a callback based system for ingestors / dispensers).
We should get rid of it, which will remove one package from this repo and make the code quite a lot easier to maintain.

Implementation Considerations

No response

Alternatives

No response

Additional information

No response

@luca-della-vedova luca-della-vedova added the enhancement New feature or request label Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant