-
Notifications
You must be signed in to change notification settings - Fork 50
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
Move to ros_control project? #28
Comments
First time to see this package for me too! |
It's currently in basic maintenance mode but would be happy to see it extended with new filters etc. There are lots of users outside of controls for example data processing. It might make more sense to create a downstream package |
I think moving it to |
I use these filters to filter pointclouds/laserscans (via laser_filters package). It wouldn't make sense to me to depend on something from the ros-control repo. |
@peci1 i think this is about moving to the ros-control org, not the ros_control repo. @davetcoleman is that right? |
You're right, I got that wrong. But it still doesn't make sense to me as I treat filters as a generally usable package. By the way, I recently released https://github.com/ctu-vras/sensor_filters which contains a set of nodes/nodelets ready-made for running filter chains on messages from sensor_msgs. |
Would still be happy to host it at @ros-controls, we are planning some controllers for ros2 with these guys |
I think it should stay here. |
I wasn't aware of this package until @gavanderhoorn pointed it out just now, but maybe this should live at https://github.com/ros-controls/ ? Thoughts @bmagyar?
The text was updated successfully, but these errors were encountered: