-
Notifications
You must be signed in to change notification settings - Fork 158
feat: add coding guideline about topic message handling #543
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
ohsawa1204
merged 229 commits into
autowarefoundation:main
from
ohsawa1204:test-add-callback-reduction-guidelines
May 17, 2024
Merged
Changes from 11 commits
Commits
Show all changes
229 commits
Select commit
Hold shift + click to select a range
2131301
doc:change 1
ohsawa1204 093bfa7
doc:change 2
ohsawa1204 6e0f248
doc:change 3
ohsawa1204 ff3014d
doc:change 4
ohsawa1204 2609785
style(pre-commit): autofix
pre-commit-ci[bot] 54f31e8
doc:change 5
ohsawa1204 cde9abf
Merge branch 'main' into test-add-callback-reduction-guidelines
ohsawa1204 cabe89b
docs(topic-message-handling): create page structure using .pages
takam5f2 13bd800
docs(topic-message-handling): fix missing links
takam5f2 418c2fa
Merge pull request #1 from takam5f2/create-page-structure-using-.pages
ohsawa1204 f38d7a0
docs(topic-message-handling): index -> index.md in 02-supp-waitset.md
ohsawa1204 7b36ae0
docs(topic-message-handling): correct misspelled words
ohsawa1204 136ab2d
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8fa4990
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 554b359
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 cba9ec8
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 564901a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 30ddb45
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 9658b88
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 64ae6cd
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 81dbcf7
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 b7a9d76
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 b94b519
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 353badd
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 2d289f1
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f7ce39b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8588e50
style(pre-commit): autofix
pre-commit-ci[bot] 98c9769
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 1c49fa3
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 2521119
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 3bb54a7
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 aa01ec7
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 22f5b2f
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 3056fd8
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 753cd0c
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5de0f78
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 7cf816e
style(pre-commit): autofix
pre-commit-ci[bot] adc814a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 fe96888
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 b786713
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a9a53e6
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 99c8db4
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a7fca6e
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 3241515
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 bb1b0be
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 537f252
style(pre-commit): autofix
pre-commit-ci[bot] 72cd561
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5e9f690
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f631391
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f81d2f9
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 2e85fe9
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 b5f2443
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 c0b3b39
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 84c293d
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 1887c69
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8e6bacf
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 508cc74
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f69cdb6
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 72a2260
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 4cf1879
style(pre-commit): autofix
pre-commit-ci[bot] 0d4d245
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 ddd6ebb
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 70bf2b4
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a82036d
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 92c51cb
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 e287933
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 1e9eeb9
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 9a32617
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 55f482a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 0cfdee9
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 77d5d03
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 89b3e47
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 b72e848
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 6d0898d
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 ab8615b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 9ca747b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8bb54f5
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 d48b903
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 6929da8
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 9d345b9
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 49fc574
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 dc01618
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 c1ad1bc
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a7f359f
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 634bc92
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 c41be37
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 0138c58
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 6627ff3
docs(topic-message-handling): some corrections related to Akamine-san…
ohsawa1204 09c6cda
Merge branch 'main' into test-add-callback-reduction-guidelines
ohsawa1204 b5a1df1
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5613357
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 4d32ce2
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 e7ca6a8
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 2855734
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 d483f9e
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 d93fcfe
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 cf82e96
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5ab6de1
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a0b8c88
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 fdbfafa
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f6aa4cd
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 710a15b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 7cd7f0b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 9487a4a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 025acb4
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 592fec9
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 1191372
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 9d00a72
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8c82a8d
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 c25e351
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 1f5889b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 4dffcab
docs(topic-message-handling): rename 02-supp-waitset.md
ohsawa1204 d00fb2e
docs(topic-message-handling): correct typo
ohsawa1204 5355a25
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 7c782d9
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 038f09f
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 bc49c9e
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 4aa951b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 94b8495
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 36c6dd5
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 c0e897b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 3e2fb08
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 af5afdd
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 6ef44d4
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 4ea3d5b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a7143bb
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 c8a779c
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 106a7e1
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 76792bb
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 6ee7031
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f71dd91
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 7937d6b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 aba3834
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8e9bf27
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a2ab1d2
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 9abdaa5
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 39e3c97
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 72123f5
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 9b5f07e
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 fb478d2
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5337eb4
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 1cd1d4a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 161623f
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 eb27439
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 b909de6
style(pre-commit): autofix
pre-commit-ci[bot] 29a6dd3
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 348f753
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 11754e2
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8043884
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 86a0891
style(pre-commit): autofix
pre-commit-ci[bot] ce33caf
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 e21c35e
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 3f5873e
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 06d1bf3
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f957d55
style(pre-commit): autofix
pre-commit-ci[bot] fb13ce0
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 41f9f19
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 29e0b33
style(pre-commit): autofix
pre-commit-ci[bot] aa50c53
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 1d082e5
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5f4faae
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 e4d4f79
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 56a0e07
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a9a84ba
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 a0410cd
style(pre-commit): autofix
pre-commit-ci[bot] e056f69
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 37a3d0a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 23606b3
style(pre-commit): autofix
pre-commit-ci[bot] e514b8a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 d644f5a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 412aaf1
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 bdc80a0
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 acc1931
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 551ade2
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 b7fc45c
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5efdab1
style(pre-commit): autofix
pre-commit-ci[bot] 1abf20f
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 060c914
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 aac9166
style(pre-commit): autofix
pre-commit-ci[bot] 9cdd072
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 6be6160
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8977a7e
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 93b8d69
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 7f15ca2
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 8c1bcb6
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 0210c99
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 51c1a66
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 817caa2
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 7bb4253
style(pre-commit): autofix
pre-commit-ci[bot] 0475034
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 426507d
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 caff81e
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 160adda
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 c4b3428
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 22e9ac8
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 c639722
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 24563bb
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 2c1589a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 aec2781
style(pre-commit): autofix
pre-commit-ci[bot] 7924922
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 d9e8972
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 66b4380
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5bac419
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 d16ccc7
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f93307a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 2e6a6ba
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 e07e307
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f6dfe4b
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 04063eb
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 57d3427
style(pre-commit): autofix
pre-commit-ci[bot] 0eda179
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 2b2100d
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 196ff01
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 71d093a
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f9a9333
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 5c5cab1
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 2ba7162
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 f30f5d8
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 fd035ab
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 78609e0
style(pre-commit): autofix
pre-commit-ci[bot] 08865bc
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 16c064b
style(pre-commit): autofix
pre-commit-ci[bot] 1740112
docs(topic-message-handling): some fixes
ohsawa1204 f7b6b0e
Merge branch 'main' into test-add-callback-reduction-guidelines
ohsawa1204 a9381c9
Update docs/contributing/coding-guidelines/ros-nodes/topic-message-ha…
ohsawa1204 7b98baa
docs(topic-message-handling): corrected as noted by mitsudome-r
ohsawa1204 d42c75e
style(pre-commit): autofix
pre-commit-ci[bot] a7bf42d
docs(topic-message-handling): minor change
ohsawa1204 47eaf3e
style(pre-commit): autofix
pre-commit-ci[bot] 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,3 +8,4 @@ nav: | |
- parameters.md | ||
- task-scheduling.md | ||
- topic-namespaces.md | ||
- topic-message-handling |
4 changes: 4 additions & 0 deletions
4
docs/contributing/coding-guidelines/ros-nodes/topic-message-handling/.pages
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
nav: | ||
- index.md | ||
- 01-supp-intra-process-comm.md | ||
- 02-supp-waitset.md | ||
71 changes: 71 additions & 0 deletions
71
...oding-guidelines/ros-nodes/topic-message-handling/01-supp-intra-process-comm.md
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,71 @@ | ||
# [supplement] Obtain a received message through intra-process communication | ||
|
||
## Topic message handling in intra-process communication | ||
|
||
rlcpp supports intra-process communication. As explained in [Topic message handling guideline](index.md), `take()` method of Subscription can not be used in case of intra-process communication. `take()` can not obtain a topic message which is received through inter-process communication. Besides, even though a method is provided to obtain a received topic message through intra-process communication, a method is not provided to refer to the message directly. | ||
But a method are provided for intra-process communication similar to a method for inter-process communication described in [obtain data by calling Subscription->take and then call a callback function](./index.md#3-obtain-data-by-calling-subscription-take-and-then-call-a-callback-function). | ||
`take_data()` method is provided to obtain a received data in case of intra-process communication and the received data must be processed through `execute()` method. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
Refer to [Template Class SubscriptionIntraProcess — rclcpp 16.0.8 documentation](http://docs.ros.org/en/humble/p/rclcpp/generated/classrclcpp_1_1experimental_1_1SubscriptionIntraProcess.html#_CPPv4N6rclcpp12experimental24SubscriptionIntraProcess9take_dataEv) for `take_data()` and `execute()` for more detail. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
## coding method | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
Call `take_data()` method and then `execute()` method as below. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
```c++ | ||
// Execute any entities of the Waitable that may be ready | ||
std::shared_ptr<void> data = waitable.take_data(); | ||
waitable.execute(data); | ||
``` | ||
|
||
Here is a sample program in [ros2_subscription_examples/intra_process_talker_listener/src/timer_listener_intra_process.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/intra_process_talker_listener/src/timer_listener_intra_process.cpp). | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
You can run the program as below. If you set `true` to `use_intra_process_comms`, intra-process communication is performed, while if you set `false`, inter-process communication is performed. | ||
|
||
```console | ||
ros2 intra_process_talker_listener talker_listener_intra_process.launch.py use_intra_process_comms:=true | ||
``` | ||
|
||
Here is a excerption from [ros2_subscription_examples/intra_process_talker_listener/src/timer_listener_intra_process.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/intra_process_talker_listener/src/timer_listener_intra_process.cpp). | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
```c++ | ||
// check if intra-process communication is enabled. | ||
if (this->get_node_options().use_intra_process_comms()){ | ||
|
||
// get the intra-process subscription's waitable. | ||
auto intra_process_sub = sub_->get_intra_process_waitable(); | ||
|
||
// check if the waitable has data. | ||
if (intra_process_sub->is_ready(nullptr) == true) { | ||
|
||
// take the data and execute the callback. | ||
std::shared_ptr<void> data = intra_process_sub->take_data(); | ||
|
||
RCLCPP_INFO(this->get_logger(), " Intra-process communication is performed."); | ||
|
||
// execute the callback. | ||
intra_process_sub->execute(data); | ||
``` | ||
|
||
Below is explanation of above code one line by one. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- `if (this->get_node_options().use_intra_process_comms()){` | ||
|
||
- verify intra-process communication is enabled or not by using NodeOptions | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- `auto intra_process_sub = sub_->get_intra_process_waitable();` | ||
takam5f2 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- get an object which is used by Subscription in case of intra-process communication | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- `if (intra_process_sub->is_ready(nullptr) == true) {` | ||
|
||
- check if a message has already been received through intra-process communication | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
- the argument of `is_ready()` is of rcl_wait_set_t type, but because the argument is not used inside `is_ready()`, `nullptr` is used tentatively | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
- using `nullptr` is a workaround at this point of time, because it is not preferable | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- `std::shared_ptr<void> data = intra_process_sub->take_data();` | ||
|
||
- obtain a topic message | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
- `intra_process_sub->take_data()` does not return Boolean value which indicates a message is received successfully or not, therefore it is needed to verify it by calling `is_ready()` in advance | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- `intra_process_sub->execute(data);` | ||
- a callback function corresponded to the received message is called inside `execute()` | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
- the callback function is executed by the thread which executes `execute()` without context switch | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
130 changes: 130 additions & 0 deletions
130
...tributing/coding-guidelines/ros-nodes/topic-message-handling/02-supp-waitset.md
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,130 @@ | ||
# [supplement] Use rclcpp::WaitSet | ||
|
||
## What is `rclcpp::WaitSet` | ||
|
||
As explained in [call take() method of Subscription object](./index.md#call-take-method-of-subscription-object), `take()` method is irreversible. Once `take()` method is executed, Subscription object state changes and undo can not be applied, therefore Subscription object can not be restored to previous state. You can use `rclcpp::WaitSet` in advance to call `take()` so that a message is received after verifying it is in Subscription Queue. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
Here is a sample code in which `wait_set_.wait()` tells you that a message has already been received and can be obtained by `take()`. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
```c++ | ||
auto wait_result = wait_set_.wait(std::chrono::milliseconds(0)); | ||
if (wait_result.kind() == rclcpp::WaitResultKind::Ready && | ||
wait_result.get_wait_set().get_rcl_wait_set().subscriptions[0]) { | ||
sub_->take(msg, msg_info); | ||
RCLCPP_INFO(this->get_logger(), "Catch message"); | ||
RCLCPP_INFO(this->get_logger(), "I heard: [%s]", msg.data.c_str()); | ||
``` | ||
|
||
Also you can verify that there are messages in multiple Subscription Queues in advance. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
If your code needs to proceed after several types of messages got together, using `rclcpp::WaitSet` is prefarable. | ||
|
||
```c++ | ||
auto wait_result = wait_set_.wait(std::chrono::milliseconds(0)); | ||
bool received_all_messages = false; | ||
if (wait_result.kind() == rclcpp::WaitResultKind::Ready) { | ||
for (auto wait_set_subs : wait_result.get_wait_set().get_rcl_wait_set().subscriptions) { | ||
if (!wait_set_subs) { | ||
RCLCPP_INFO_THROTTLE(get_logger(), clock, 5000, "Waiting for data..."); | ||
return {}; | ||
} | ||
} | ||
received_all_mesages = true; | ||
} | ||
``` | ||
|
||
In code above, unless `rclcpp::WaitSet` is used, it is difficult to verify that all needed messages get together. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
## Coding method | ||
|
||
We will explain coding method of `rclcpp::WaitSet` using a sample code below. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- [ros2_subscription_examples/waitset_examples/src/talker_triple.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/waitset_examples/src/talker_triple.cpp) | ||
- it publishes `/chatter` per one second, `/slower_chatter` per two seconds, and `/slowest_chatter` per three seconds periodically | ||
- [ros2_subscription_examples/waitset_examples/src/timer_listener_triple_async.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/waitset_examples/src/timer_listener_triple_async.cpp) | ||
- it queries `WaitSet` per one second and if there is a message available, it obtains the message by `take()` | ||
- it has each subscription for `/chatter` `/slower_chatter`, and `/slower_chatter` | ||
|
||
Following three steps are needed to use `WaitSet`. | ||
|
||
### 1. declare and initialize `WaitSet` | ||
|
||
You need to declare WaitSet variable first to use. | ||
Below is excerption from [ros2_subscription_examples/waitset_examples/src/timer_listener_triple_async.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/waitset_examples/src/timer_listener_triple_async.cpp). | ||
|
||
```c++ | ||
rclcpp::WaitSet wait_set_; | ||
``` | ||
|
||
Note that there are three types of `WaitSet` as below. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
- [Typedef rclcpp::WaitSet](https://docs.ros.org/en/ros2_packages/humble/api/rclcpp/generated/typedef_namespacerclcpp_1ad6fb19c154de27e92430309d2da25ac3.html) | ||
- Subscription, Timer, and so on can be registered to WaitSet at any time | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
- [Typedef rclcpp::ThreadSafeWaitSet](https://docs.ros.org/en/ros2_packages/humble/api/rclcpp/generated/typedef_namespacerclcpp_1acaec573e71549fd3078644e18e7f7127.html) | ||
- Subscription, Timer, and so on can be registered to WaitSet only in thread-safe state | ||
- sample code is here: [examples/rclcpp/wait_set/src/thread_safe_wait_set.cpp at rolling · ros2/examples](https://github.com/ros2/examples/blob/rolling/rclcpp/wait_set/src/thread_safe_wait_set.cpp) | ||
- [Typedef rclcpp::StaticWaitSet](https://docs.ros.org/en/ros2_packages/humble/api/rclcpp/generated/typedef_namespacerclcpp_1adb06acf4a5723b1445fa6ed4e8f73374.html) | ||
- Subscription, Timer, and so on can be registered to WaitSet only at initialization | ||
- here are sample codes: | ||
- [ros2_subscription_examples/waitset_examples/src/timer_listener_twin_static.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/waitset_examples/src/timer_listener_twin_static.cpp) | ||
- [examples/rclcpp/wait_set/src/static_wait_set.cpp at rolling · ros2/examples](https://github.com/ros2/examples/blob/rolling/rclcpp/wait_set/src/static_wait_set.cpp) | ||
|
||
### 2. register trigger (Subscription, Timer, and so on) to `WaitSet` | ||
|
||
You need to register a trigger to `WaitSet`. | ||
Below is excerption from [ros2_subscription_examples/waitset_examples/src/timer_listener_triple_async.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/waitset_examples/src/timer_listener_triple_async.cpp) | ||
|
||
```c++ | ||
subscriptions_array_[0] = create_subscription<std_msgs::msg::String>("chatter", qos, not_executed_callback, subscription_options); | ||
subscriptions_array_[1] = create_subscription<std_msgs::msg::String>("slower_chatter", qos, not_executed_callback, subscription_options); | ||
subscriptions_array_[2] = create_subscription<std_msgs::msg::String>("slowest_chatter", qos, not_executed_callback, subscription_options); | ||
|
||
// Add subscription to waitset | ||
for (auto & subscription : subscriptions_array_) { | ||
wait_set_.add_subscription(subscription); | ||
} | ||
``` | ||
|
||
In code above, created subscriptions are registered to `WaitSet` by `add_subscription()`. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
You can also register another type of trigger to WaitSet by which a callback function registered to Subscription is invoked, such as Timer, Service, or Action. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
A sample code to register Timer trigger is here. | ||
|
||
```c++ | ||
wait_set_.add_timer(much_slower_timer_); | ||
``` | ||
|
||
A trigger can be registered at declaration and initialization as [https://github.com/ros2/examples/blob/rolling/rclcpp/wait_set/src/wait_set_topics_and_timer.cpp#L66](https://github.com/ros2/examples/blob/rolling/rclcpp/wait_set/src/wait_set_topics_and_timer.cpp#L66). | ||
|
||
### 3. verify WaitSet result | ||
|
||
You can see WaitSet result by doing 1 below first and then 2 below. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
1. verifying if any trigger has been invoked | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
2. verifying if specified trigger has been invoked | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
As for 1, here is a sample code excerpted from [ros2_subscription_examples/waitset_examples/src/timer_listener_triple_async.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/waitset_examples/src/timer_listener_triple_async.cpp). | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
```c++ | ||
auto wait_result = wait_set_.wait(std::chrono::milliseconds(0)); | ||
if (wait_result.kind() == rclcpp::WaitResultKind::Ready) { | ||
RCLCPP_INFO(this->get_logger(), "wait_set tells that some subscription is ready"); | ||
} else { | ||
RCLCPP_INFO(this->get_logger(), "wait_set tells that any subscription is not ready and return"); | ||
return; | ||
} | ||
``` | ||
|
||
In code above, it is verified whether any trigger has been invoked which is registered to `wait_set_` by `auto wait_result = wait_set_.wait(std::chrono::milliseconds(0))`. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
if `wait_result.kind() == rclcpp::WaitResultKind::Ready` is `true`, it indicates any trigger has been invoked. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
As for 2, here is a sample code excerpted from [ros2_subscription_examples/waitset_examples/src/timer_listener_triple_async.cpp at main · takam5f2/ros2_subscription_examples](https://github.com/takam5f2/ros2_subscription_examples/blob/main/waitset_examples/src/timer_listener_triple_async.cpp). | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
```c++ | ||
for (size_t i = 0; i < subscriptions_num; i++) { | ||
if (wait_result.get_wait_set().get_rcl_wait_set().subscriptions[i]) { | ||
std_msgs::msg::String msg; | ||
rclcpp::MessageInfo msg_info; | ||
if (subscriptions_array_[i]->take(msg, msg_info)) { | ||
RCLCPP_INFO(this->get_logger(), "Catch message via subscription[%ld]", i); | ||
RCLCPP_INFO(this->get_logger(), "I heard: [%s]", msg.data.c_str()); | ||
``` | ||
|
||
In code above, `wait_result.get_wait_set().get_rcl_wait_set().subscriptions[i]` indicates whether each individual trigger has been invoked or not. The result is stored to `subscriptions` array. The order in `subscriptions` array is the same as the order in which triggers are registered. | ||
ohsawa1204 marked this conversation as resolved.
Show resolved
Hide resolved
|
Oops, something went wrong.
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.