Investigate replacing all NaNs everywhere

[gusgordon]

Currently, we replace all NaNs in cum_returns. This makes the logic more robust, but might not alert us to certain data errors. We will want to check if there are any cases where there are NaNs in the return stream and how it makes sense to handle that.

We may want to raise an error or warning in case this happens.