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

MAINT Fix gross leverage - round 2 #147

Merged
merged 1 commit into from
Sep 26, 2015
Merged

MAINT Fix gross leverage - round 2 #147

merged 1 commit into from
Sep 26, 2015

Conversation

humdings
Copy link
Contributor

This is separate branch that changes the long_short_exposure
function to match gross leverage.

#140 is the same thing, but my git repo
got out of whack while working on the other branch so I figured I'd start again.

This only changes the code in pyfolio, I did not re-run the notebook
since it seemed to cause an issue last time. I'll submit the notebook
change in a separate PR.

This is separate branch that changes the `long_short_exposure`
function to match gross leverage.

My git repo got out of whack while working on the other branch
so I figured I'd start again

This only changes the code in pyfolio, I did not re-run the notebook
since it seemed to cause an issue last time. I'll submit the notebook
change in a separate PR.
@humdings
Copy link
Contributor Author

Are there any objections to merging this?

@justinlent
Copy link
Contributor

ship it!

Sent from iPhone

On Fri, Sep 25, 2015 at 5:11 PM -0700, "David Edwards" [email protected] wrote:

Are there any objections to merging this?


Reply to this email directly or view it on GitHub.

twiecki added a commit that referenced this pull request Sep 26, 2015
MAINT Fix gross leverage - round 2
@twiecki twiecki merged commit 32ffa2f into master Sep 26, 2015
@twiecki twiecki deleted the gross-leverage branch September 26, 2015 10:17
@twiecki
Copy link
Contributor

twiecki commented Sep 26, 2015

Thanks @humdings!

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

Successfully merging this pull request may close these issues.

3 participants