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

implement proper lazy subscribers #119

Closed
mikeferguson opened this issue Feb 22, 2023 · 4 comments
Closed

implement proper lazy subscribers #119

mikeferguson opened this issue Feb 22, 2023 · 4 comments
Assignees

Comments

@mikeferguson
Copy link
Member

This is blocked by upstream work that should be included in Iron turtle - ros2/rmw#330 (comment)

This was referenced Feb 22, 2023
@130s 130s mentioned this issue Feb 23, 2023
4 tasks
@mikeferguson mikeferguson self-assigned this May 17, 2023
@mikeferguson
Copy link
Member Author

Started on this - it is an epic rabbit hole:

  • We need to update image_proc/depth_proc to make this work (they aren't lazy, and they subscribe to our outputs)
  • That means we need to update image_transport (which most of the image_proc subscribers use)
  • That might also lead to changing things in message_filters, which image_transport uses (but so far, maybe not).

@mikeferguson
Copy link
Member Author

Work in Progress:

@mikeferguson
Copy link
Member Author

As a note - this will NOT make it into Iron, since we can't get the image_common changes merged into Iron (apparently that package is part of the base and was frozen on April 17). I will continue to get this merged into rolling, but it won't make into a regular release until 2024

@mikeferguson
Copy link
Member Author

image_pipeline is now updated - but not yet released

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

1 participant