Simplify metric init and remove cyclic dependency in Metric and Store

Metrics take a "ResultStore" in init, but they only really use it to update the name of the metric outputs... that should be simply removed and done explicitly. Right now Store imports Metric and vice-versa, which is not ideal.