[CM] Improve memory allocation with buffer variables

[saikishor]

This PR tries to address the issue of memory allocation in the RT loop of the CM read, write, and update cycles

[codecov]

Codecov Report

Attention: Patch coverage is 97.14286% with 1 line in your changes missing coverage. Please review.

Project coverage is 87.95%. Comparing base (23bd1c3) to head (52dda97).

Files with missing lines	Patch %	Lines
.../include/controller_manager/controller_manager.hpp	92.30%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1801      +/-   ##
==========================================
- Coverage   87.97%   87.95%   -0.03%     
==========================================
  Files         121      121              
  Lines       12403    12385      -18     
  Branches     1105     1101       -4     
==========================================
- Hits        10912    10893      -19     
  Misses       1085     1085              
- Partials      406      407       +1     

Flag	Coverage Δ
unittests	87.95% <97.14%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
controller_manager/src/controller_manager.cpp	77.55% <100.00%> (-0.56%)	⬇️
.../include/controller_manager/controller_manager.hpp	96.77% <92.30%> (-3.23%)	⬇️

---- 🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests

[fmauch]

This seems fair. Is there a reason that you went for a quite large number to reserve memory for?

[gavanderhoorn]

Question: would using an RT-safe memory allocator not be a more scalable approach?

[mergify]

This pull request is in conflict. Could you fix it @saikishor?

[saikishor]

This seems fair. Is there a reason that you went for a quite large number to reserve memory for?

Good question, the part of the vectors I can reduce it. For the part of the string, I think we will need it, when you have a large setup and it helps when concatenating strings

[saikishor]

Question: would using an RT-safe memory allocator not be a more scalable approach?

@gavanderhoorn Sure, I wanted to keep it simple and isolated, so it's easier for others. No big reason apart from that. Let me know what you think

[bmagyar]

Question: would using an RT-safe memory allocator not be a more scalable approach?

Do you have some pointers for us? (no pun intended! :D )