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

Added topic coherence rst files #906

Merged
merged 2 commits into from
Sep 30, 2016
Merged

Conversation

devashishd12
Copy link
Contributor

@devashishd12 devashishd12 commented Sep 30, 2016

Referring to issue #905. @tmylk just a small addition here.

@devashishd12
Copy link
Contributor Author

I'll add the topic coherence dir as well here.

@devashishd12
Copy link
Contributor Author

@tmylk ready for merge.

@devashishd12 devashishd12 changed the title Added coherencemodel.rst Added topic coherence rst files Sep 30, 2016
@tmylk
Copy link
Contributor

tmylk commented Sep 30, 2016

Thanks for taking this on! Did you check that html files appear after running the commands in https://github.com/RaRe-Technologies/gensim/wiki/Developer-page#documentation?

@devashishd12
Copy link
Contributor Author

oh getting an error there:

Exception occurred:
  File "/home/devashish/EXPERIMENTATION/gensim/docs/src/./gensim_theme/genindex.html", line 49, in block "body"
    {%- for entryname, (links, subitems, _) in column %}
ValueError: need more than 2 values to unpack
The full traceback has been saved in /tmp/sphinx-err-eVoHAI.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!
Makefile:34: recipe for target 'html' failed
make: *** [html] Error 1

@devashishd12
Copy link
Contributor Author

updated to Sphinx 1.4 and now it builds fine. Sorry for the noise!

@tmylk tmylk merged commit 99b5977 into piskvorky:develop Sep 30, 2016
@tmylk
Copy link
Contributor

tmylk commented Sep 30, 2016

Thanks for the PR!

@devashishd12
Copy link
Contributor Author

No problem! The topic coherence docs look hideous currently. Will improve on that soon.

@devashishd12 devashishd12 deleted the coherence_rst branch September 30, 2016 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants