Mnaveau/time measurement bug - Githubissues

machines-in-motion / dynamic_graph_manager

BSD 3-Clause "New" or "Revised" License

4 stars 4 forks source link

Mnaveau/time measurement bug #17

Closed MaximilienNaveau closed 4 years ago

MaximilienNaveau commented 4 years ago

Description

In this PR I:

Fixed an issue related to the time measurement buffer size from an optional YAML paramter
Fixed the C++ formatting
Fixed some warning.
Try to force the warning to display themselves.

How I Tested

I ran the unit-tests on my machines

I fulfilled the following requirements

[x] All new code is formatted according to our style guide (for C++ run clang-format, for Python, run flake8 and fix all warnings).
[x] All new functions/classes are documented and existing documentation is updated according to changes.
[x] No commented code from testing/debugging is kept (unless there is a good reason to keep it).