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

ENH: Put a cache in front of future chain lookups #1455

Merged
merged 1 commit into from Aug 31, 2016

Conversation

Projects
None yet
3 participants
@jbredeche
Member

jbredeche commented Aug 31, 2016

Cache the last 100 rootsymbol/session pairs, since future chains never
change inside a session.

ENH: Put a cache in front of future chain lookups
Cache the last 100 rootsymbol/session pairs, since future chains never
change inside a session.
@ehebert

This comment has been minimized.

Member

ehebert commented Aug 31, 2016

LGTM

@jbredeche jbredeche merged commit 35631f4 into master Aug 31, 2016

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jbredeche jbredeche deleted the cache-future-chain-lookups branch Aug 31, 2016

@coveralls

This comment has been minimized.

coveralls commented Aug 31, 2016

Coverage Status

Coverage increased (+0.002%) to 86.143% when pulling f570ab0 on cache-future-chain-lookups into 34b113f on master.

bartosh pushed a commit to bartosh/zipline that referenced this pull request Sep 27, 2016

Merge pull request quantopian#1455 from quantopian/cache-future-chain…
…-lookups

ENH: Put a cache in front of future chain lookups
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment