-
Notifications
You must be signed in to change notification settings - Fork 199
add content-filtered-topic interfaces #894
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
Merged
iuhilnehc-ynos
merged 34 commits into
ros2:master
from
iuhilnehc-ynos:topic-content_filtered_topic
Apr 4, 2022
Merged
Changes from 30 commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
2016557
to support a feature of content filtered topic
593a4bf
Update function description
fc651ec
Nit.
1af2e83
Update based on review
953ad14
Not to reset error if subscrption is invalid.
9626a6c
remove copy function for subscription_options
da7be13
update comments to make linelength <= 100
14ff61e
Update comments
0aaf6e1
add test for filter data by all cft interfaces with rcl_take
190948c
update interface
6381b08
update test
154ed6f
add lost test file
680d0c2
update test case
069b196
nit
328e8ff
fix for unsupported cft and unscrutify
65f53dd
fix unscrutify
f3364d9
rename
5550615
refactor test
bdc50e6
fix for uncrustify and typo
1667f3a
relate to `rcutils_string_array_t expression_parameters` changed in rmw
e03e121
add necessary structure for fallback interfaces
100d4a0
remove the implementation temporary, add them with fallback in the fe…
b33d745
address comments
0ca5105
update comments
3f6421f
add DDS content filter implementation without fallback
eaa8727
waiting to allow for filter propagation
cd8e859
use = instead of match symbol as it is not standard
0fcdc5b
remove unnecessary code and update error message
206facd
update test case name and use BasicTypes
05811f0
use the rcl_subscription_option_t allocator instead
d421cf2
set the option data directly
6301133
explicitly check the cft supported by rmw_connextdds and rmw_fastrtps…
4934583
increase the maximun time for events and content filter propagation
954982e
set test timeout to 120
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.