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] zed_interfaces as a package? #750

Closed
flabrosse opened this issue Jul 15, 2021 · 14 comments
Closed

[Feature] zed_interfaces as a package? #750

flabrosse opened this issue Jul 15, 2021 · 14 comments
Assignees

Comments

@flabrosse
Copy link

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

@Myzhar
Copy link
Member

Myzhar commented Jul 15, 2021

@flabrosse zed_interfaces is already on a separate package, it's in the same Github repository, but the ROS packages are all separated.
You can simply copy the folder zed_interfaces in the src folder of the catkin workspace of the second computer and start to compile to make everything work.

@flabrosse
Copy link
Author

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

@davesarmoury
Copy link

+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

@Myzhar
Copy link
Member

Myzhar commented Jul 21, 2021

I see what I can do about this request 👍

@Myzhar
Copy link
Member

Myzhar commented Jul 26, 2021

The zed_interfaces package has been removed from this repository and moved to its own zed-ros-interfaces repository for allowing better integration of the ZED Wrapper on remote ground stations that do not require the full package to be installed. To update your repository please follow the new update instructions.

@Myzhar Myzhar closed this as completed Jul 26, 2021
@davesarmoury
Copy link

@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

@Myzhar
Copy link
Member

Myzhar commented Jul 26, 2021

Yes, it makes sense because they are loaded by Rviz and not propagated with the "robot description".

@Myzhar
Copy link
Member

Myzhar commented Jul 26, 2021

@davesarmoury done. Now the meshes are in the zed_interfaces repository and they can be correctly loaded by Rviz

$ git pull --recurse-submodules # update recursively all the submodules and it's for you

@davesarmoury
Copy link

davesarmoury commented Jul 26, 2021 via email

@Myzhar
Copy link
Member

Myzhar commented Jul 26, 2021

Thank you for your feedback

@flabrosse
Copy link
Author

Thank you indeed!

@flabrosse
Copy link
Author

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...)

@Myzhar
Copy link
Member

Myzhar commented Aug 4, 2021

Because that's not the name of the package, but the name of the Github repository.
The names of the ROS packages included in this repository follow the ROS naming standard:
zed_ros -> the meta-package
zed_wrapper -> the package that contains the ZED Wrapper
zed_ros_interfaces -> the sub-module including the definitions of the custom topics and services
zed_nodelet -> the package that contains the ZED Nodelets

@flabrosse
Copy link
Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants