micro-ROS / rmw_microxrcedds

RMW implementation using Micro XRCE-DDS middleware.
Apache License 2.0
30 stars 23 forks source link

Spin session on empty wait #233

Closed pablogs9 closed 2 years ago

pablogs9 commented 2 years ago

Related: https://github.com/eProsima/Micro-XRCE-DDS-Client/pull/316

github-actions[bot] commented 2 years ago

Static memory analysis

Default configuration

MTU: 512 B Input buffer size: 2048 B Input history: 4 Output buffer size: 2048 B Output history: 4

Entity Qty Size per unit
Context 2 5616 B
Topic 8 56 B
Service 4 248 B
Client 4 248 B
Subscription 4 272 B
Publisher 4 288 B
Node 4 208 B
Static input buffer 8 2136 B
Init options 6 64 B
Wait sets 4 56 B
Guard Condition 4 64 B

TOTAL: 34468 B

codecov-commenter commented 2 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (galactic@51942f6). Click here to learn what that means. The diff coverage is n/a.

:exclamation: Current head e308776 differs from pull request most recent head 16c2c15. Consider uploading reports for the commit 16c2c15 to get more accurate results

@@             Coverage Diff             @@
##             galactic     #233   +/-   ##
===========================================
  Coverage            ?   50.08%           
===========================================
  Files               ?       43           
  Lines               ?     1687           
  Branches            ?      480           
===========================================
  Hits                ?      845           
  Misses              ?      619           
  Partials            ?      223           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 51942f6...16c2c15. Read the comment docs.

github-actions[bot] commented 2 years ago

Static memory analysis

Default configuration

MTU: 512 B Input buffer size: 2048 B Input history: 4 Output buffer size: 2048 B Output history: 4

Entity Qty Size per unit
Context 2 5616 B
Topic 8 56 B
Service 4 248 B
Client 4 248 B
Subscription 4 272 B
Publisher 4 288 B
Node 4 208 B
Static input buffer 8 2136 B
Init options 6 64 B
Wait sets 4 56 B
Guard Condition 4 64 B

TOTAL: 34468 B

pablogs9 commented 2 years ago

@mergify backport main foxy

mergify[bot] commented 2 years ago

backport main foxy

🟠 Waiting for conditions to match

- [ ] `merged` [:pushpin: backport requirement]

Hey, I reacted but my real name is @Mergifyio

mergify[bot] commented 2 years ago

backport main foxy

✅ Backports have been created

* [#238 Spin session on empty wait (backport #233)](https://github.com/micro-ROS/rmw_microxrcedds/pull/238) has been created for branch `main` * [#239 Spin session on empty wait (backport #233)](https://github.com/micro-ROS/rmw_microxrcedds/pull/239) has been created for branch `foxy`