Avoid things like:

Use of uninitialized value $die in concatenation (.) or string at /home/atheurer/repos/crucible/subprojects/tools/procstat//procstat-post-process line 111

I think we can just populate with '0' when we don't have this topo info.

Metadata is quite wrong in some places

Metrics created with log_sample(), where all of the values are 0, get de-duped to a single value of 0, then eliminated (by default). In some cases, eliminating the metric completely is not a good idea. For example, the metric is a primary-metric for a benchmark, and the metric is required, even if it is "0".

To fix this, finish_samples() which is always required to be called once all samples have been logged, can take an optional argument, when this argument value is 1, will not eliminate the metric (if its value is 0).