-
Notifications
You must be signed in to change notification settings - Fork 41
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
implicit BPR-MF wrapper model #132
Comments
Created an experimental PR. #232 I know there are several discussion points, but I hope it clarifies the desired specs. Currently, the PR excludes training with features. I think support fit with features for GPU is easier than CPU because preparation before |
@chezou thank you so much for your contributions! I've merged PR with BPR model. I wouldn't focus on adding features for BPR since it's a complicated task and will take a lot of time. Right now in RecTools we are focusing on adding features and providing maximum customisation to transformer models (SASRec and BERT4Rec). They can be found here: https://github.com/MobileTeleSystems/RecTools/blob/experimental/sasrec/examples/tutorials/transformers_tutorial.ipynb Another big story right now is CandidateRankingModel which uses baseline models to generate candidates and then uses Gradient Boosting to rerank them. https://github.com/MobileTeleSystems/RecTools/blob/experimental/two_stage/examples/tutorials/candidate_ranking_model_tutorial.ipynb As for the baselines, we still need SLIM model. It is well known both for quality and efficiency. And I do have a feeling that we are missing it in the framework right now. We also have an issue for it: #103 |
Thanks for the review and comment. I don't have any preference for having the feature for BPR, so feel free to close this issue. I briefly researched SLIM (specifically, SLIM ElasticNet) and found that many frameworks use this CPU-based implementation: https://github.com/MaurizioFD/RecSys2019_DeepLearning_Evaluation/blob/master/SLIM_ElasticNet/SLIMElasticNetRecommender.py, or some use a PyTorch-based one. Anyway, let's continue to discuss on #103 |
Feature Description
Create a wrapper for popular BPR-MF baseline
Why this feature?
It's easy to implement and it's often used as a baseline in research
Additional context
No response
The text was updated successfully, but these errors were encountered: