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

Naming mismatch for LogisticMatrixFactorization and LogisticMF #32

Open
omgwenxx opened this issue Feb 12, 2025 · 0 comments
Open

Naming mismatch for LogisticMatrixFactorization and LogisticMF #32

omgwenxx opened this issue Feb 12, 2025 · 0 comments

Comments

@omgwenxx
Copy link

omgwenxx commented Feb 12, 2025

Describe the bug
There is a mismatch in the naming for LogisticMatrixFactorization. Using LogisticMatrixFactorization as stated in the docs returns an AttributeError

Image

Using LogisticMF returns a TypeError due to the naming mismatch with the class (I assume)

Image

Using this config:

experiment:
  backend: pytorch
  binarize: true
  data_config:
    strategy: fixed
    test_path: ./hm_splits_test/test_indexed.tsv
    train_path: ./hm_splits_test/train_indexed.tsv
  dataset: hm
  evaluation:
    cutoffs:
    - 5
    - 12
    simple_metrics:
    - nDCG
    - MAP
  gpu: 0
  models:
    LogisticMF:
      alpha: 0.5
      batch_size: 512
      epochs: 10
      factors: 10
      lr: 0.001
      meta:
        save_recs: true
      reg: 0.1
  top_k: 12

BR,
Gwen

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

No branches or pull requests

1 participant