From a4ea3e3343245ff187c1904c6b0063f29d38abda Mon Sep 17 00:00:00 2001 From: fawce Date: Wed, 30 May 2012 23:55:25 -0400 Subject: [PATCH] marked a todo for a calculation bug --- zipline/finance/risk.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/zipline/finance/risk.py b/zipline/finance/risk.py index f4bd736f..b4888f9e 100644 --- a/zipline/finance/risk.py +++ b/zipline/finance/risk.py @@ -187,6 +187,8 @@ class RiskMetrics(): return period_returns, returns def calculate_volatility(self, daily_returns): + # TODO: we should be using an annualized number for the + # square root, not the days in the period. return np.std(daily_returns, ddof=1) * math.sqrt(self.trading_days) def calculate_sharpe(self):