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

adding net leverage, long/short exposure, long/short position count #464

Merged
merged 4 commits into from Jan 8, 2015

Conversation

Projects
None yet
3 participants
@fawce
Member

fawce commented Jan 7, 2015

to performance packets.

@twiecki

This comment has been minimized.

Contributor

twiecki commented Jan 7, 2015

zipline/finance/performance/period.py:349:80: E501 line too long (80 > 79 characters)

@twiecki

This comment has been minimized.

Contributor

twiecki commented Jan 7, 2015

We should also add the new leverage features to the release docs.

@twiecki

This comment has been minimized.

Contributor

twiecki commented Jan 8, 2015

👍

@fawce

This comment has been minimized.

Member

fawce commented Jan 8, 2015

@twiecki I updated the release notes. let me know if I did it right :).

@fawce

This comment has been minimized.

Member

fawce commented Jan 8, 2015

passed performance review with @ehebert so merge time

fawce added a commit that referenced this pull request Jan 8, 2015

Merge pull request #464 from quantopian/expand_perf_packet
adding net leverage, long/short exposure, long/short position count

@fawce fawce merged commit ec055b6 into master Jan 8, 2015

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details

@fawce fawce deleted the expand_perf_packet branch Jan 8, 2015

@coveralls

This comment has been minimized.

coveralls commented Jan 8, 2015

Coverage Status

Coverage increased (+0.01%) when pulling 893e8be on expand_perf_packet into 6a41faf on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment