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

Clean up references to Morfessor, tox and gensim.models.wrappers #3345

Merged
merged 3 commits into from
May 5, 2022
Merged

Clean up references to Morfessor, tox and gensim.models.wrappers #3345

merged 3 commits into from
May 5, 2022

Commits on May 5, 2022

  1. Drop reference to Morfessor 

    It was dropped from usage in 2022.

Fixes: commit acce8a2
    @pabs3
    pabs3 committed May 5, 2022
    Configuration menu
    Copy the full SHA
    d70d42d View commit details
    Browse the repository at this point in the history

  2. Drop references to tox 

    Usage of tox was dropped in 2022.

Where necessary, replace tox commands with the current equivalent.

Fixes: commit 3ce81a4
    @pabs3
    pabs3 committed May 5, 2022
    Configuration menu
    Copy the full SHA
    c2d1d15 View commit details
    Browse the repository at this point in the history

  3. Drop direct and indirect references to gensim.modules.wrapper(s) 

    They were all removed in 2021.

TestDtmModel escaped being noticed via test failures because all the tests
it runs are skipped without the DTM_PATH environment variable being set.

Fixes: commit a21d9cc
    @pabs3
    pabs3 committed May 5, 2022
    Configuration menu
    Copy the full SHA
    4f67af1 View commit details
    Browse the repository at this point in the history