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.

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

Fix documentation for gensim.corpora.dictionary and gensim.corpora.hashdictionary. Partial fix #1671 #1814

Merged
merged 16 commits into from
Jan 22, 2018

Conversation

CLearERR
Copy link
Contributor

@CLearERR CLearERR commented Dec 24, 2017

Is a child PR for #1671
@menshikh-iv

@menshikh-iv menshikh-iv added the incubator project PR is RaRe incubator project label Dec 26, 2017
@menshikh-iv menshikh-iv changed the title Refactor API reference gensim.corpora: dictionary.py & hashdictionary.py Fix documentation for gensim.corpora.dictionary and `gensim.corpora.hashdictionary. Partial fix #1671 Jan 18, 2018
@menshikh-iv menshikh-iv changed the title Fix documentation for gensim.corpora.dictionary and `gensim.corpora.hashdictionary. Partial fix #1671 Fix documentation for gensim.corpora.dictionary and gensim.corpora.hashdictionary. Partial fix #1671 Jan 18, 2018
@menshikh-iv menshikh-iv merged commit 09672b3 into piskvorky:develop Jan 22, 2018
sj29-innovate pushed a commit to sj29-innovate/gensim that referenced this pull request Feb 21, 2018
….hashdictionary`. Partial fix piskvorky#1671 (piskvorky#1814)

* Refactored Dictionary description

* Some additions to dictionary.py

* More docstrings

* Docstringed dictionary.py (except from_corpus method)

* Added docstrings for hashdictionary.py

* Fix for numbered lists

* Merged branch

* fix hashdictionary[1]

* fix hashdictionary[2]

* fix hashdictionary[3]

* fix dictionary[1]

* fix hashdictionary[4]

* fix dictionary[2]

* fix dictionary[3]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incubator project PR is RaRe incubator project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants