Feature/ipm with rclcpp serialized messages

[DensoADAS]

Enabling publishing rclcpp::SerializedMessage for IPM (rcl_serialized_message_t are not supported for IPM).

Generic publisher can be created now with rclcpp::Publisherrclcpp::SerializedMessage. Therefore GenericPublisher is not needed anymore.

uses changes from: https://github.com/ros2/rclcpp/pull/1928

[DensoADAS]

updated to review

[DensoADAS]

I think I processed all comments

[DensoADAS]

while testing I found two bugs which I fixed

[DensoADAS]

@alsora are further steps required from us? thank you

[DensoADAS]

What is missing for approval? Thank you :smile:

[alsora]

Hi @DensoADAS, this is a 1-year old PR and it looks like it has been completely rebased and/or reworked. It will require a full review from scratch before proceeding.

I'll try to do it this week.

[DensoADAS]

Hi @DensoADAS, this is a 1-year old PR and it looks like it has been completely rebased and/or reworked. It will require a full review from scratch before proceeding.

I'll try to do it this week.

@alsora Did you have already time for the review? Thank you.

[DensoADAS]

Hi @DensoADAS, this is a 1-year old PR and it looks like it has been completely rebased and/or reworked. It will require a full review from scratch before proceeding.

I'll try to do it this week.

any updates?

[DensoADAS]

@ivanpauno @hidmic @wjwwood

Could someone also review the code? Thank you

[DensoADAS]

Any updates from the reviewers?

[DensoADAS]

rebased

[alsora]

Sorry for the wait. CI build:

• Linux
• Linux-aarch64
• Windows

[alsora]

The PR looks good. If possible, I would ask to add a few more unit-tests to validate the changes in the intra-process manager

[DensoADAS]

I will add some unit tests