Skip to content
This repository was archived by the owner on Apr 14, 2022. It is now read-only.

probablepeople library causes LS to peg a CPU core #877

Closed
karthiknadig opened this issue Apr 3, 2019 · 1 comment
Closed

probablepeople library causes LS to peg a CPU core #877

karthiknadig opened this issue Apr 3, 2019 · 1 comment
Assignees
Milestone

Comments

@karthiknadig
Copy link
Member

@jakebailey figured out that this is caused by the large number of dictionary entries in that library

@jakebailey jakebailey self-assigned this Apr 3, 2019
@jakebailey jakebailey added this to the April 2019.1 milestone Apr 3, 2019
@jakebailey
Copy link
Member

Blocked on #881, since I'll need to set IsExact to false with #886 when I limit the dictionary size.

jakebailey added a commit that referenced this issue Apr 9, 2019
Fixes #877.

Limit the size of exact collections to 1000 items (when they are created from the expressions). This is set high in order to not affect potentially large `__all__` definitions.

probablepeople has a dictionary with more than 150,000 key/value pairs, which takes a very long time to process; 1000 doesn't take very long.
jakebailey added a commit to jakebailey/python-language-server that referenced this issue Nov 1, 2019
Fixes microsoft#877.

Limit the size of exact collections to 1000 items (when they are created from the expressions). This is set high in order to not affect potentially large `__all__` definitions.

probablepeople has a dictionary with more than 150,000 key/value pairs, which takes a very long time to process; 1000 doesn't take very long.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants