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

Support gensim4 LdaModel #73

Merged
merged 15 commits into from
Feb 5, 2024
Merged

Support gensim4 LdaModel #73

merged 15 commits into from
Feb 5, 2024

Commits on Jan 4, 2024

  1. Reflect renamed module for LdaModel in gensim4.

    `LdaModel`, that was in `gensim.models.lda`(gensim3), is implemented by `gensim.models.ldamodel` now (gensim4). The proposed solution tries both and exits with an error if neither module can be imported.
    larsgrobe authored Jan 4, 2024
    Configuration menu
    Copy the full SHA
    3bb0548 View commit details
    Browse the repository at this point in the history

Commits on Jan 5, 2024

  1. Support gensim4 ldamodel

    Check wether version 3 or 4 of gensim is loaded and train the lda model accordingly.
    larsgrobe authored Jan 5, 2024
    Configuration menu
    Copy the full SHA
    ed4c14d View commit details
    Browse the repository at this point in the history
  2. Minor clean-up

    Just added a missing space.
    larsgrobe authored Jan 5, 2024
    Configuration menu
    Copy the full SHA
    98b7eb5 View commit details
    Browse the repository at this point in the history
  3. Import sys.exit() to raise error if gensim not available

    This error-handling may not comply with the general litstudy convention.
    larsgrobe authored Jan 5, 2024
    Configuration menu
    Copy the full SHA
    bcec9c0 View commit details
    Browse the repository at this point in the history
  4. Fix gensim version check

    Convert the mayor version number to int as expected in the conditional.
    larsgrobe authored Jan 5, 2024
    Configuration menu
    Copy the full SHA
    b739626 View commit details
    Browse the repository at this point in the history
  5. Pass num_topics to LdaModel

    Pass the num_topics parameter to LdaModel with gensim4.
    larsgrobe authored Jan 5, 2024
    Configuration menu
    Copy the full SHA
    db995c1 View commit details
    Browse the repository at this point in the history
  6. Fix train_lda_model()

    Added the missing '.T'.
    larsgrobe authored Jan 5, 2024
    Configuration menu
    Copy the full SHA
    6f43d34 View commit details
    Browse the repository at this point in the history

Commits on Jan 31, 2024

  1. Configuration menu
    Copy the full SHA
    53903a6 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    0495363 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    452a4a6 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    1cefc82 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    8bc42b2 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    28b3324 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    ffbf7d4 View commit details
    Browse the repository at this point in the history
  8. Set default algorithm back to ldamulticore, which requires to define …

    …a main() function in the calling module.
    larsgrobe committed Jan 31, 2024
    Configuration menu
    Copy the full SHA
    48f42ea View commit details
    Browse the repository at this point in the history