search

micro-ROS / rmw_microxrcedds

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

Add timing API for handling XRCE-DDS session #153

Closed pablogs9 closed 3 years ago

pablogs9 commented 3 years ago

Before merge: https://github.com/eProsima/Micro-XRCE-DDS-Client/pull/260

github-actions[bot] commented 3 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 1 5592 B
Topic 12 56 B
Service 4 80 B
Client 4 80 B
Subscription 4 168 B
Publisher 8 184 B
Node 4 56 B
Static input buffer 8 2120 B

TOTAL: 25060 B

codecov-commenter commented 3 years ago

Codecov Report

Merging #153 (4bbf3c0) into main (1ec1489) will decrease coverage by 0.40%. The diff coverage is 34.78%.

:exclamation: Current head 4bbf3c0 differs from pull request most recent head 4864738. Consider uploading reports for the commit 4864738 to get more accurate results Impacted file tree graph

@@            Coverage Diff             @@
##             main     #153      +/-   ##
==========================================
- Coverage   47.88%   47.47%   -0.41%     
==========================================
  Files          41       42       +1     
  Lines        1510     1527      +17     
  Branches      304      307       +3     
==========================================
+ Hits          723      725       +2     
- Misses        579      594      +15     
  Partials      208      208
Impacted Files Coverage Δ
rmw_microxrcedds_c/src/rmw_microros/timing.c 0.00% <0.00%> (ø)
rmw_microxrcedds_c/src/types.c 58.94% <ø> (ø)
rmw_microxrcedds_c/src/rmw_client.c 54.54% <100.00%> (+0.46%) :arrow_up:
rmw_microxrcedds_c/src/rmw_publish.c 47.61% <100.00%> (+1.27%) :arrow_up:
rmw_microxrcedds_c/src/rmw_publisher.c 49.62% <100.00%> (+0.37%) :arrow_up:
rmw_microxrcedds_c/src/rmw_request.c 65.38% <100.00%> (ø)
rmw_microxrcedds_c/src/rmw_response.c 64.70% <100.00%> (ø)
rmw_microxrcedds_c/src/rmw_service.c 53.53% <100.00%> (-0.47%) :arrow_down:

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 1ec1489...4864738. Read the comment docs.

github-actions[bot] commented 3 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 1 5592 B
Topic 8 56 B
Service 4 80 B
Client 4 80 B
Subscription 4 168 B
Publisher 4 184 B
Node 4 56 B
Static input buffer 8 2120 B

TOTAL: 25052 B

github-actions[bot] commented 3 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 1 5592 B
Topic 8 56 B
Service 4 80 B
Client 4 80 B
Subscription 4 168 B
Publisher 4 184 B
Node 4 56 B
Static input buffer 8 2120 B

TOTAL: 25052 B

pablogs9 commented 3 years ago

@mergify backport foxy galactic

mergify[bot] commented 3 years ago

Command backport foxy galactic: pending

Waiting for the pull request to get merged

Hey, I reacted but my real name is @Mergifyio

github-actions[bot] commented 3 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 1 5592 B
Topic 8 56 B
Service 4 80 B
Client 4 80 B
Subscription 4 168 B
Publisher 4 184 B
Node 4 56 B
Static input buffer 8 2120 B

TOTAL: 25052 B

mergify[bot] commented 3 years ago

Command backport foxy galactic: failure

No backport have been created GitHub App like Mergify are not allowed to create pull request where .github/workflows is changed.