Skip to content

added documentation about number_of_handles in all examples.#341

Merged
JanStaschulat merged 3 commits intomasterfrom
feature/simplify-examples
Mar 10, 2023
Merged

added documentation about number_of_handles in all examples.#341
JanStaschulat merged 3 commits intomasterfrom
feature/simplify-examples

Conversation

@JanStaschulat
Copy link
Copy Markdown
Contributor

@JanStaschulat JanStaschulat commented Mar 2, 2023

  • added a comment to possibly modify XRCE-DDS configuration regarding total number of handles in all examples
  • removed rclc_executor_prepare() function in basic examples (as it is optional)

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>
@JanStaschulat JanStaschulat self-assigned this Mar 2, 2023
@JanStaschulat JanStaschulat requested a review from pablogs9 March 2, 2023 13:25
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #341 (2efedd4) into master (3b4ff9e) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #341      +/-   ##
==========================================
- Coverage   69.20%   69.13%   -0.08%     
==========================================
  Files          16       16              
  Lines        2715     2715              
  Branches      765      765              
==========================================
- Hits         1879     1877       -2     
- Misses        450      451       +1     
- Partials      386      387       +1     
Impacted Files Coverage Δ
rclc/src/rclc/action_goal_handle.c 59.87% <0.00%> (-1.28%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>
Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>
@JanStaschulat
Copy link
Copy Markdown
Contributor Author

@mergify backport humble foxy

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Mar 2, 2023

backport humble foxy

✅ Backports have been created

Details

@JanStaschulat
Copy link
Copy Markdown
Contributor Author

@pablogs9 could you have a look?

Copy link
Copy Markdown
Member

@pablogs9 pablogs9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@JanStaschulat JanStaschulat merged commit 810afe1 into master Mar 10, 2023
@JanStaschulat JanStaschulat deleted the feature/simplify-examples branch March 10, 2023 12:36
mergify bot pushed a commit that referenced this pull request Mar 10, 2023
* added documentation about number_of_handles in all examples.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

* aligned prepare() and spin() in all functions

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

* removed 'prepare' functions - to make it easier for beginners.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

---------

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>
(cherry picked from commit 810afe1)
mergify bot pushed a commit that referenced this pull request Mar 10, 2023
* added documentation about number_of_handles in all examples.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

* aligned prepare() and spin() in all functions

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

* removed 'prepare' functions - to make it easier for beginners.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

---------

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>
(cherry picked from commit 810afe1)

# Conflicts:
#	rclc_examples/src/example_action_client.c
#	rclc_examples/src/example_action_server.c
#	rclc_examples/src/example_lifecycle_node.c
#	rclc_examples/src/example_pingpong.cpp
JanStaschulat added a commit that referenced this pull request Mar 22, 2023
…342)

* added documentation about number_of_handles in all examples.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

* aligned prepare() and spin() in all functions

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

* removed 'prepare' functions - to make it easier for beginners.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

---------

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>
(cherry picked from commit 810afe1)

Co-authored-by: Jan Staschulat <jan.staschulat@de.bosch.com>
JanStaschulat added a commit that referenced this pull request Mar 31, 2023
…#341) (#343)

* added documentation about number_of_handles in all examples. (#341)

* added documentation about number_of_handles in all examples.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

* aligned prepare() and spin() in all functions

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

* removed 'prepare' functions - to make it easier for beginners.

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

---------

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>
(cherry picked from commit 810afe1)

# Conflicts:
#	rclc_examples/src/example_action_client.c
#	rclc_examples/src/example_action_server.c
#	rclc_examples/src/example_lifecycle_node.c
#	rclc_examples/src/example_pingpong.cpp

* resolved merge conflicts

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>

---------

Signed-off-by: Jan Staschulat <jan.staschulat@de.bosch.com>
Co-authored-by: Jan Staschulat <jan.staschulat@de.bosch.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants