Skip to content

Conversation

@theisjendal
Copy link
Contributor

Description

I have added a new model named HypAR, a new example file for HypAR, and made edits to Lightgcn.

For HypAR I've created the recom_hyper file, requirements.txt and requirements_cu116.txt.
The example file runs the method on the Cellphone dataset. As this is not part of Cornac, I simply define the methods for creating the dataset directly in the example. The parameters for HypAR are found on the validation set.

For LightGCN I have (see commit 1799bf8):

  • Removed normalization for layers, not used for lgcn.
  • Fixed sum weight constant to num layers instead of cur layer index.
  • Allow lgcn to take blocks.
  • Fixed requirement error caused by newer dgl versions.
  • Moved edge normalization to graph for easier use.
    Without these LightGCN wouldn't perform as required.

Todo: Add reference to paper once publicised in readme.

Checklist:

  • I have added tests.
  • I have updated the documentation accordingly.
  • I have updated README.md (if you are adding a new model).
  • I have updated examples/README.md (if you are adding a new example).
  • I have updated datasets/README.md (if you are adding a new dataset).

@lthoang
Copy link
Member

lthoang commented Mar 13, 2024

LGTM

@lthoang lthoang requested a review from qtuantruong March 13, 2024 14:29
@qtuantruong
Copy link
Member

@theisjendal @lthoang thanks for creating and reviewing this PR. Since there are many changes introduced, I would suggest to create separate PR that is dedicated to the changes in the LightGCN model first. I also would like to learn the motivation behind those changes. After that, we can revisit this PR for the purpose of adding the HypAR model. How does that sound?

@theisjendal theisjendal mentioned this pull request Mar 14, 2024
5 tasks
@darrylong darrylong added the models New models, changes to models label Mar 18, 2024
@lthoang lthoang requested a review from qtuantruong March 18, 2024 10:56
@theisjendal theisjendal requested a review from qtuantruong March 19, 2024 08:58
@qtuantruong
Copy link
Member

LGTM

@lthoang lthoang merged commit 44a8fc9 into PreferredAI:master Mar 20, 2024
@theisjendal theisjendal mentioned this pull request Mar 23, 2024
5 tasks
qtuantruong pushed a commit to qtuantruong/cornac that referenced this pull request Oct 25, 2025
…referredAI#600)

* Init method

* Example file

* Added HypAR

Added hyper and its functions.

* Added dataset standard files and virtual env names to ignore

* Create graph comments

* Flock comments

* Graph wrapper comments

* ao embedding comments

* Remove learned ui embeddings

* Learned embedding and norm comments

* fit comments

* Removed unused code

* Comments on main file

* Removed more and comments

* More comments

* More comments

* Updated readmes

* Fix dataset name

* Reference fix

* Path fix

* Removed example

* Removed HypAR from examples readme

* Styling

* Added hypar doc string

* Removed spacing

* Added paper link

* Moved docstrings to earlier

* Simplified model type

* Added paper reference

* Added lightgcn as part of HypAR

* Updated url

* Fix double normalize of first layer
qtuantruong pushed a commit to qtuantruong/cornac that referenced this pull request Oct 25, 2025
…referredAI#600)

* Init method

* Example file

* Added HypAR

Added hyper and its functions.

* Added dataset standard files and virtual env names to ignore

* Create graph comments

* Flock comments

* Graph wrapper comments

* ao embedding comments

* Remove learned ui embeddings

* Learned embedding and norm comments

* fit comments

* Removed unused code

* Comments on main file

* Removed more and comments

* More comments

* More comments

* Updated readmes

* Fix dataset name

* Reference fix

* Path fix

* Removed example

* Removed HypAR from examples readme

* Styling

* Added hypar doc string

* Removed spacing

* Added paper link

* Moved docstrings to earlier

* Simplified model type

* Added paper reference

* Added lightgcn as part of HypAR

* Updated url

* Fix double normalize of first layer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

models New models, changes to models

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants