Skip to content

Commit

Permalink
BUG: Issue #154 Inconsistent "Returns Analysis" Table
Browse files Browse the repository at this point in the history
The "Mean Period Wise Returns, Top Quantile and Bottom Quantile" rows are quoted as a holding period excess return (i.e. ending_value/beginning_value - 1). However, the "Mean Period Wise Spread (bps)" row is quoted on a one-period basis (i.e. it uses the utils.rate_of_return function to convert holding period return to a one period rate of return).

All the three rows now use  one-period notation
  • Loading branch information
MichaelJMath authored and luca-s committed Jun 19, 2017
1 parent 2aa3efc commit a9a47aa
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions alphalens/tears.py
Expand Up @@ -101,7 +101,7 @@ def create_summary_tear_sheet(factor_data, long_short=True):

plotting.plot_quantile_statistics_table(factor_data)

plotting.plot_returns_table(alpha_beta, mean_ret_quantile, mean_ret_spread_quant)
plotting.plot_returns_table(alpha_beta, mean_compret_quantile, mean_ret_spread_quant)

plotting.plot_quantile_returns_bar(mean_compret_quantile,
by_group=False,
Expand Down Expand Up @@ -175,7 +175,7 @@ def create_returns_tear_sheet(factor_data, long_short=True, by_group=False):
vertical_sections = 2 + fr_cols * 3
gf = GridFigure(rows=vertical_sections, cols=1)

plotting.plot_returns_table(alpha_beta, mean_ret_quantile, mean_ret_spread_quant)
plotting.plot_returns_table(alpha_beta, mean_compret_quantile, mean_ret_spread_quant)

plotting.plot_quantile_returns_bar(mean_compret_quantile,
by_group=False,
Expand Down

0 comments on commit a9a47aa

Please sign in to comment.