Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix formula in Mallet documentation #2186

Merged
merged 3 commits into from
Sep 18, 2018
Merged

Fix formula in Mallet documentation #2186

merged 3 commits into from
Sep 18, 2018

Conversation

Laubeee
Copy link
Contributor

@Laubeee Laubeee commented Sep 14, 2018

@menshikh-iv
Copy link
Contributor

@Laubeee hello, thanks for PR, please check documentation build log and fix issue https://circleci.com/gh/RaRe-Technologies/gensim/1502?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link, after I'll merge your PR

@Laubeee
Copy link
Contributor Author

Laubeee commented Sep 17, 2018

hey, circle CI is fixed, dunno why travis is now failing, my change has nothing to do with the FastTextModel..

@menshikh-iv
Copy link
Contributor

menshikh-iv commented Sep 17, 2018 •
edited
Loading

@Laubeee I re-run Travis (this is not-deterministic tests), don't worry about it.
Though issue with rendering still here (but new)
image

@menshikh-iv menshikh-iv changed the title Fix docstring Fix formula in Mallet documentation Sep 18, 2018
@menshikh-iv
Copy link
Contributor

Thanks for PR @Laubeee 👍

@menshikh-iv menshikh-iv merged commit 4224879 into piskvorky:develop Sep 18, 2018
@Laubeee Laubeee deleted the documentation-fix branch September 18, 2018 15:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Laubeee @menshikh-iv