Michael Carroll

> For what it is worth, I also tried to reproduce this locally, and I can't do it. I attempted to reproduce this with the `timer_test` packages from the original...

We are going to need more information about which platform and versions you are using.

I don't recall there ever being a design doc around dynamic reconfigure. At this point, I would consider the implementation a reference implementation. I believe as a server, at a...

> The approach used in the core libraries does not handle the case that CMAKE_CXX_STANDARD is defined, but an empty value. While I understand that this case isn't handled, I'm...

In theory, you can pack whatever distortion model you want between the string field and the distortion parameters. It's not clear to me why that wouldn't be sufficient here?

Can you describe the bug that this is fixing please?

This one likely needs another pass with test failures.

> At least, the uninitialized variable issue should be merged, this is a bugfix. Do you want a separate PR? Yes please, I have no issues with that part. I'll...

The `foxy` branch doesn't have support for services, only topics. You may be able to backport the functionality or use a newer branch (possibly with some dependency tweaks).

I made a baseline comparison for reading and converting all of the events in a trace file: https://gist.github.com/mjcarroll/34e7f06d761c8c6ce2cce36027900b34 The main conclusion is that using babeltrace2 via their python API (https://babeltrace.org/docs/v2.0/python/bt2/)...