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.

Sign up for GitHub

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

Jump to bottom

Dramatic Memory Usage Increases After Pandas 18 Merge #1503

Closed
ssanderson opened this issue Sep 21, 2016 · 2 comments
Closed

Dramatic Memory Usage Increases After Pandas 18 Merge #1503

ssanderson opened this issue Sep 21, 2016 · 2 comments
Labels
Testing

Comments

@ssanderson
Copy link
Contributor

Prior to the merge of #1339, peak memory usage of the test suite was around 800MB. Post merge, we peak at over 2GB on my machine.
ssanderson pushed a commit that referenced this issue Sep 21, 2016
PERF: Remove or defer calls to get_loc on large indices.
e86fffc 
Mitigation for #1503.
@ssanderson ssanderson mentioned this issue Sep 21, 2016
Merged
@ssanderson
Copy link
Contributor Author

ssanderson commented Sep 21, 2016 •
edited
Loading

Root cause appears to be pandas-dev/pandas#14266

@freddiev4
Copy link
Contributor

freddiev4 commented Mar 11, 2017 •
edited
Loading

Does the PR you merged fix that issue? Or rather, what's the status on this?

Just curious in regards to the recent release we did

fredfortier pushed a commit to scrtlabs/catalyst that referenced this issue Aug 14, 2017
PERF: Remove or defer calls to get_loc on large indices.
a9faf94 
Mitigation for quantopian/zipline#1503.
vikram-narayan pushed a commit to quantopian/trading_calendars that referenced this issue Jun 12, 2018
PERF: Remove or defer calls to get_loc on large indices.
915383e 
Mitigation for quantopian/zipline#1503.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ssanderson @ernestoeperez88 @freddiev4