-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[FEATURE] Replace SASRec TF with PyTorch version #2111
Comments
|
solved with pip install cvxpy another error:
solved by installing install feather-format |
|
Work so far: staging...miguel/sasrec_unirec Next step is to create a unit test called The steps should follow the structure of https://github.com/recommenders-team/recommenders/blob/main/examples/00_quick_start/sar_movielens.ipynb:
if we want, we can also do:
After this, we will create a notebook explaining an end 2 end case with a real dataset, and we will replace the TF notebook. |
Description
SASRec tests are disabled: https://github.com/recommenders-team/recommenders/blob/main/tests/ci/azureml_tests/test_groups.py#L410
We could replace the TF algo with https://github.com/microsoft/UniRec/blob/main/unirec/model/sequential/sasrec.py
Expected behavior with the suggested feature
Branch: https://github.com/recommenders-team/recommenders/tree/miguel/sasrec_unirec
Tasks:
Other Comments
The text was updated successfully, but these errors were encountered: