Any reason this couldn't be shortened to simply ```name```? Just to reduce a little of the redundancy so that something like:

```MetricFu::Metric.metrics.map(&:metric_name)``` becomes ```MetricFu::Metric.metrics.map(&:name)```
 and
```metric.metric_name``` simplifies to ```metric.name```