Conversation
Signed-off-by: Karsten Knese <Karsten1987@users.noreply.github.com>
|
@Karsten1987 this might be a bit beyond my depth but i wouldn't mind trying to help out here. it looks like you had a direction you were going and just need someone to implement. more than happy to help if you point me in the right direction |
|
@flynneva thanks for picking this up. The basic idea here is as follows: IMAGE_TRANSPORT_PUBLIC
CameraPublisher(
rclcpp::Node * node,
const std::string & base_topic,
rmw_qos_profile_t custom_qos = rmw_qos_profile_default);While this makes quite a bit of sense, it prohibits though to initialize it with a IMAGE_TRANSPORT_PUBLIC
CameraPublisher(
rclcpp::node_interfaces::NodeBaseInterface::SharedPtr node_base_interface,
rclcpp::node_interfaces::NodeTopicsInterface::SharedPtr node_topics_interface,
rclcpp::node_interfaces::NodeLoggingInterface::SharedPtr node_logging_interface,
const std::string & base_topic,
rmw_qos_profile_t custom_qos = rmw_qos_profile_default);
template<class NodeT>
CameraPublisher(
NodeT && node,
const std::string & base_topic,
rmw_qos_profile_t custom_qos = rmw_qos_profile_default)
: CameraPublisher(
node->get_node_base_interface(),
node->get_node_topics_interface(),
node->get_node_logging_interface(),
base_topic, custom_qos)That's basically what I started here. It's a bit tedious do to do and touches quite a bit of files along the way. |
|
@Karsten1987 sounds great and thank you for the more in-depth information. I had already started to work towards this and will pick it up later this week. it does seem like a lot of work but I think should be done in order to make these tools usable for anyone who wants to use lifecycle nodes. my plan right now is to first help out with the uncrustify issues for the |
|
I have added lifecycle support to |
|
@Rayman I won't really work on this in the near future. It might be easier for you to open a new PR - incorporate my existing commits from this PR if helpful - and I'll close out this PR. |
|
Is there any progress on this topic? |
|
is it still stalled ? |
|
As @Karsten1987 commented here #167 (comment) he will not put more effort in this PR. If someone want to reopen a new PR feel free to do it. Happy to review it |
closes #108
Currently still in draft mode. Call for participation.
The current architecture of image transport is pretty complex for easily integrating lifecycle nodes as
rclcpp::Nodesare passed along pretty heavily. Hence, all functions are basically overloaded to deal with their individual node interfaces to avoid templating.