micro-ROS / rmw_microxrcedds

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

Refactor micro-ROS custom APIs #103

Closed pablogs9 closed 3 years ago

pablogs9 commented 3 years ago

Related:

TODO:

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 5584 B
Topic 12 56 B
Service 4 80 B
Client 4 72 B
Subscription 4 168 B
Publisher 8 184 B
Node 4 56 B
Static input buffer 8 2120 B

TOTAL: 25020 B

codecov-commenter commented 3 years ago

Codecov Report

Merging #103 (179411b) into foxy (a3db073) will not change coverage. The diff coverage is 40.95%.

Impacted file tree graph

@@           Coverage Diff           @@
##             foxy     #103   +/-   ##
=======================================
  Coverage   48.01%   48.01%           
=======================================
  Files          35       39    +4     
  Lines        1487     1487           
  Branches      303      303           
=======================================
  Hits          714      714           
  Misses        567      567           
  Partials      206      206           
Impacted Files Coverage Δ
rmw_microxrcedds_c/src/rmw_init.c 63.20% <ø> (ø)
rmw_microxrcedds_c/src/rmw_microros/init_options.c 0.00% <0.00%> (ø)
rmw_microxrcedds_c/src/rmw_publish.c 46.34% <ø> (ø)
rmw_microxrcedds_c/src/rmw_microros/time_sync.c 40.62% <40.62%> (ø)
rmw_microxrcedds_c/src/rmw_microros/discovery.c 65.00% <65.00%> (ø)
rmw_microxrcedds_c/src/rmw_microros/ping.c 78.57% <78.57%> (ø)
...cedds_c/src/rmw_microros/continous_serialization.c 100.00% <100.00%> (ø)
rmw_microxrcedds_c/src/rmw_uxrce_transports.c 75.00% <100.00%> (ø)
... and 3 more

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 a3db073...179411b. 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 5584 B
Topic 12 56 B
Service 4 80 B
Client 4 72 B
Subscription 4 168 B
Publisher 8 184 B
Node 4 56 B
Static input buffer 8 2120 B

TOTAL: 25020 B

pablogs9 commented 3 years ago

@Acuadros95 is possible to remove eProsimas references from the Doxygen? I guess that with the default template should be ok.

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 5584 B
Topic 12 56 B
Service 4 80 B
Client 4 72 B
Subscription 4 168 B
Publisher 8 184 B
Node 4 56 B
Static input buffer 8 2120 B

TOTAL: 25020 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 5584 B
Topic 12 56 B
Service 4 80 B
Client 4 72 B
Subscription 4 168 B
Publisher 8 184 B
Node 4 56 B
Static input buffer 8 2120 B

TOTAL: 25020 B