It seems there are situations where you can end up with an asset with zero activity, in which case we get NaN for the LCOX value (see discussion in #1115). I thought we were checking for NaNs in metrics, but seemingly not. It could break things, so we should handle this case properly.
