Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cumulative_returns calculation no longer adjusts for periods > 1? #386

Open
jeromeku opened this issue Sep 9, 2020 · 0 comments
Open

cumulative_returns calculation no longer adjusts for periods > 1? #386

jeromeku opened this issue Sep 9, 2020 · 0 comments

Comments

@jeromeku
Copy link

jeromeku commented Sep 9, 2020

Problem Description

It seems that the plot_cumulative_returns_by_quantile function in the performance module is no longer adjusting for period > 1. I was under the impression that the cumulative returns are calculated as explained in issue #187 and here on the Quantopian forum. However looking at the latest master branch definition of this function shows that the period parameter is only being used to label the plot and NOT in the cumulative returns definition, which is calculated using the empyrical cum_returns function which calculated simple daily returns.

Please provide a minimal, self-contained, and reproducible example:

[Paste code here]

Please provide the full traceback:

[Paste traceback here]

Please provide any additional information below:

Versions

  • Alphalens version: master
  • Python version:
  • Pandas version:
  • Matplotlib version:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant