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

Count_occurrences doesn't correctly track regular expressions #40

Open
OliverSherouse opened this issue Feb 26, 2018 · 0 comments
Open

Count_occurrences doesn't correctly track regular expressions #40

OliverSherouse opened this issue Feb 26, 2018 · 0 comments
Assignees
@jnelson16
Labels
bug

Comments

@OliverSherouse
Copy link
Contributor

quantgov/quantgov/corpora/builtins.py

Line 123 in 5c22c6e

+ tuple(term_counts[i] for i in terms)

We need the term_counts dictionary keys to be the actual regexes, not the string matches. We should map the one to the other and then generates results from the remapped version.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jnelson16 jnelson16

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@OliverSherouse @jnelson16