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

RTAudio ROS Package Split #6

Open
elaske opened this issue Dec 5, 2016 · 1 comment
Open

RTAudio ROS Package Split #6

elaske opened this issue Dec 5, 2016 · 1 comment

Comments

@elaske
Copy link

elaske commented Dec 5, 2016

It might be nice to make this part of the package a standalone ROS wrapper for RTAudio / ROS fork for it.

It looks like the code is just added to this repo and tracked separately. At the very least, it'd be a good idea to submodule the real code so that you don't have that tracked separately.

It will make upgrading it a lot easier in the future if either of these things are done.

@francoisferland
Copy link
Member

Yes, that was the original plan (as it used to be part of another, much larger repository), since rt_audio_ros does not have any dependencies on manyears_* packages, and we had applications were rt_audio_ros was used without sound source localization.

They remained in the same repository for practical deployment issues at the time, but now they could obviously be in their own trees.

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

2 participants