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

support zero-copy services #15

Open
protobits opened this issue Apr 14, 2014 · 7 comments
Open

support zero-copy services #15

protobits opened this issue Apr 14, 2014 · 7 comments

Comments

@protobits
Copy link

As far as I undertand, zero-copy is not implemented for services. It would be really useful to have it. For example, if I want to share a large data structure that does not change, it makes more sense to provide a service that allows to retrieve it and store the reference. Otherwise, I'm forced to publish it periodically, which does not make sense.

@dirk-thomas dirk-thomas added this to the untargeted milestone Apr 14, 2014
@dirk-thomas
Copy link
Member

While this would be a great feature there is currently no feature development planned for nodelet_core. Without external contributions this improvement is not likely to be addressed by the maintainers. Therefore I will mark the ticket with the milestone untargeted.

That being said we will definitely consider this as an interesting feature for the future development of ROS.

@protobits
Copy link
Author

OK. If I get to understand the innards of nodelet/ROS I might try to
contribute it, but I can't ensure this.
However, may I ask why nodelet will not be developed in the near future? Is
this intra-process model not a priority for ROS? I consider this aspect of
ROS a really important feature (for example, for computer-vision tasks
where serializing images is out of the question).
Or is it simply a feature freeze?

On Mon, Apr 14, 2014 at 2:23 AM, Dirk Thomas [email protected]:

While this would be a great feature there is currently no feature
development planned for nodelet_core. Without external contributions this
improvement is not likely to be addressed by the maintainers. Therefore I
will mark the ticket with the milestone untargeted.

That being said we will definitely consider this as an interesting feature
for the future development of ROS.


Reply to this email directly or view it on GitHubhttps://github.com//issues/15#issuecomment-40334089
.

@dirk-thomas
Copy link
Member

Most of the core ROS packages are in "maintained" status rather then being actively developed with new features planned. This is because we (at OSRF) have only a limited amount of time to work on ROS and we decided that it is better spend on the next iteration of ROS. While we still fix major issues with ROS we can't spend a significant amount of work on new features for existing ROS packages. But any pull requests are welcome and will be reviewed / integrated.

@protobits
Copy link
Author

Ok, thank you for your response. I will see if I manage to include this
functionality. However, I don't have a very good understanding on how
nodelet achieves its functionality regarding avoiding serialization. I will
ask a question on answers.ros.org about it.

On Mon, Apr 14, 2014 at 2:18 PM, Dirk Thomas [email protected]:

Most of the core ROS packages are in "maintained" status rather then being
actively developed with new features planned. This is because we (at OSRF)
have only a limited amount of time to work on ROS and we decided that it is
better spend on the next iteration of ROS. While we still fix major issues
with ROS we can't spend a significant amount of work on new features for
existing ROS packages. But any pull requests are welcome and will be
reviewed / integrated.


Reply to this email directly or view it on GitHubhttps://github.com//issues/15#issuecomment-40392855
.

@tfoote
Copy link
Member

tfoote commented Apr 14, 2014

Note: this is actually a feature request for ros/ros_comm/roscpp not nodelet_core. When you go digging that's where to look. https://github.com/ros/ros_comm

@protobits
Copy link
Author

Should I open an issue there then and close this?

@dirk-thomas
Copy link
Member

I think the ticket here is perfectly fine since it only applies to nodelets (even if the enhancement ends up needing changes in roscpp).

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

No branches or pull requests

4 participants