Skip to content

Support for SubscriptionOptions::ignore_local_publications#537

Closed
jamoralp wants to merge 10 commits intoros2:galacticfrom
eProsima:feature/ignore_local_pub
Closed

Support for SubscriptionOptions::ignore_local_publications#537
jamoralp wants to merge 10 commits intoros2:galacticfrom
eProsima:feature/ignore_local_pub

Conversation

@jamoralp
Copy link
Copy Markdown
Contributor

Signed-off-by: Jose Antonio Moral joseantoniomoralparras@gmail.com

christophebedard and others added 5 commits May 4, 2021 10:25
Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
)

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Added is_plain_ attribute to base TypeSupport.
* Added new methods to base TypeSupport.
* Implementation of rmw_borrow_loaned_message.
* Implementation of rmw_return_loaned_message_from_publisher.
* Enable loan messages on publishers of plain types.
* Implementation for taking loaned messages.
* Enable loan messages on subscriptions of plain types.

Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Co-authored-by: Michel Hidalgo <michel@ekumenlabs.com>
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
@hidmic
Copy link
Copy Markdown

hidmic commented May 31, 2021

@jamoralp let's get this patch reviewed and merged into Rolling (#536) first. We can backport it with ROS Boss approvial later on.

Barry-Xu-2018 and others added 3 commits June 2, 2021 13:42
Signed-off-by: Barry Xu <barry.xu@sony.com>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
@clalancette clalancette added the more-information-needed Further information is required label Jun 17, 2021
jamoralp added 2 commits June 18, 2021 07:12
Signed-off-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
Signed-off-by: Jose Antonio Moral <joseantoniomoralparras@gmail.com>
@MiguelCompany MiguelCompany force-pushed the feature/ignore_local_pub branch from b3096b9 to 116ecfb Compare June 18, 2021 05:13
@MiguelCompany MiguelCompany deleted the feature/ignore_local_pub branch June 22, 2021 10:04
@MiguelCompany
Copy link
Copy Markdown
Collaborator

@hidmic Please close this in favor of #543. @jamoralp left eProsima and I don't know if he could close it or not.

@hidmic hidmic closed this Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

more-information-needed Further information is required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants