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

Move to ros_control project? #28

Open
davetcoleman opened this issue Jul 22, 2019 · 8 comments
Open

Move to ros_control project? #28

davetcoleman opened this issue Jul 22, 2019 · 8 comments

Comments

@davetcoleman
Copy link

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?

@bmagyar
Copy link

bmagyar commented Jul 23, 2019

First time to see this package for me too!
I'd be up for hosting this in that org, no problem. It seems rather low-bandwidth at the moment, are you guys planning on extending it?

@tfoote
Copy link
Member

tfoote commented Aug 2, 2019

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 control_filters that provides specific filters for controls applications as plugins that can be used in the filter chains provided by this package.

@davetcoleman
Copy link
Author

I think moving it to ros-control would provide a lot more visibility and usage, and prevent potential duplicate effort, but I have no immediate needs or use cases.

@peci1
Copy link
Contributor

peci1 commented Jun 24, 2020

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.

@jonbinney
Copy link
Contributor

@peci1 i think this is about moving to the ros-control org, not the ros_control repo. @davetcoleman is that right?

@peci1
Copy link
Contributor

peci1 commented Jul 2, 2021

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.

@bmagyar
Copy link

bmagyar commented Jul 2, 2021

Would still be happy to host it at @ros-controls, we are planning some controllers for ros2 with these guys

@DLu
Copy link

DLu commented Jan 13, 2024

I think it should stay here.

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

No branches or pull requests

6 participants