- Sponsor
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
replace all occurences of docFreq
with dfs
#9
Comments
Ok why not, go ahead. |
here you go my friend: |
Thanks again Dieter, I applied the changes and pushed them to Btw I thought that closes the github pull request automatically, but I can still see it. What gives? Do I have to close it manually? EDIT: I applied the patch with |
not sure. it might. I suggest you add me as a remote, fetch my branch, diff it, and then merge it (or rebase on top of it if you don't want a merge commit), then push to github. |
Ok, I'll try that on the next pull. |
most of the gensim codebase uses names like
dfs
andidfs
,but dictionary still has an attribute docFreq, which is exactly the same thing as a
dfs
dictionary.I suggest to replace all occurrences of
docFreq
withdfs
The text was updated successfully, but these errors were encountered: