-
Notifications
You must be signed in to change notification settings - Fork 392
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] zed_interfaces as a package? #750
Comments
@flabrosse |
Hi @Myzhar, thanks again for the quick response. Yes, this is what I ended up doing, but this also means that if I want to do a git pull, the whole thing will be pulled again. Not the end of the world, just annoying. Thanks |
+1 for this request. Having interfaces and description packages in their own repo would make a distributed application easier. This repo can even include the new repo as a git sub module |
I see what I can do about this request 👍 |
The |
@Myzhar The meshes from https://github.com/stereolabs/zed-ros-wrapper/tree/master/zed_wrapper/urdf/models should also be moved so you can use rviz remotely without getting errors |
Yes, it makes sense because they are loaded by Rviz and not propagated with the "robot description". |
@davesarmoury done. Now the meshes are in the
|
Wonderful! Small improvement that will make life much easier :)
|
Thank you for your feedback |
Thank you indeed! |
Any particular reason why the package is called zed-ros-interfaces rather than zed_ros_interfaces? The latter is the ROS convention (I myself are in the process of renaming things...) |
Because that's not the name of the package, but the name of the Github repository. |
Fair enough. I always find it confusing that the top directory is not called the same as the package, but I have seen this before when there is more than one package. |
I'm trying to use services to talk to the ZED from a different computer than where the ZED is connected (and where the zed.launch is started). This implies having zed_interfaces on this computer. As far as I can see, the only way to have that is to install the whole of the ros wrapper.
Could we have a separate package for zed_interfaces?
This is made even more needed since on the latest ubuntu (20.04), the install fails because the package python-geographiclib is now python3-geographiclib.
Cheers,
Fred
The text was updated successfully, but these errors were encountered: