-
Notifications
You must be signed in to change notification settings - Fork 118
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
[RMP] Tensorflow support for session based recommendations integration in Merlin #433
Comments
@sararb @gabrielspmoreira can you flesh this out as best as possible in @marcromeyn's absence. |
@gabrielspmoreira what does the architecture look like for the system for session based? Are we planning to use session generation to feed into a candidate generation stage? |
@
@EvenOldridge the session-based recommendation works as a next-item prediction task. It can be seen as a retrieval model, where the query tower users a sequential model (e.g. RNN, Transformer) and outputs a query representation/vector. During inference, such vectors can be used to retrieve the similar items from ANN the same way a retrieval model does. |
@karlhigley , is the systems section here updated. Pleaes review |
It is up to date with the current state of our knowledge of the work |
@marcromeyn , in one of the meetings, I made a note that this task is dependent on some tasks covered in RMP479-EMBEDDINGS initiative [RMP] Enable users to pass embedding tables directly into the input block in order to more easily support new functionality (non-trainable embeddings, different dimensions, model parallel, etc) . Is this correct ? which are these tasks. ? @EvenOldridge for vis. |
@rnyak , please link the systems - multi hot related development |
@oliverholworthy , please add the input output schema related tickets to this ticket |
@viswa-nvidia For the saving method: This is the parent issue for that: We have implemented the save method to save input schema, but currently missing output schema. |
Also identified an error in Merlin Models that may impact ability to serve Transformer-based models that affects issues with saving a model after loading. |
@karlhigley , please add the Triton related PR ( serving signatures haven't matched up with what model expects ) in the ticket. |
@rnyak to follow up with @radekosmulski for blocker ( 23.04 ) |
It was pointed out to me that we should be consistent about when we consider something done or not so I'm going to reopen this and move it to 22.05. @bbozkaya You've had the only remaining ticket (review the API) on your todo for the past two weeks with no progress. Is this something you're able to take on so that we can close the ticket. If not let us know and we can reassign. |
Problem:
Session-based and sequential-based models are an active research area for providing personalized recommendations. Transformers4Rec library was built to support the definition of such architectures and the results of our experiments conducted in the T4Rec paper showed the effectiveness of Transformers in modeling short sequences observed in session-based tasks. The T4Rec library was also used to win various RecSys challenges. We also observe a growing interest and active engagement from customers in using Transformers4Rec.
T4Rec was not actively updated for several months as the team shifted its focus to developing the Merlin Model library. MM does not currently support sequential and session-based recsys architectures. MM should support these sequential architectures and provide all necessary support to our users so that they can build such effective models.
Goal:
Definition of Done
Have an example that serves TF session based model in conjunction with a NVT workflow where the session based models scores the whole catalog
Constraints:
- Two out of the 4 masking classes are missing: PLM and RTD
- Support for training techniques embedding in HuggingFace trainer class: multi-gpu, early stopping, checkpoints saving...
- Conduct experiments with real-world datasets (like the one conducted in the T4Rec paper with the Pytorch API)
Starting Point:
Training
Proposed API:
Inputs
Masking
RetrievalModel
Outputs
Port Sequence Architectures
Fixes for the GTC tutorial on session-based recommendation
Inference support
Save schema on model save
Session-base model can be used as a candidate generation model, for that we need the following options:
Define NextITemPredictionTask as a sub-class of RetrievalModel.Note: Make sure we can export the encoder block (Transformer or RNN) together with a SequenceSummary post layer as the query tower. models#736
Note: Make sure we can export the encoder block (Transformer or RNN) together with a SequenceSummary post layer as the query tower.
Export the item embeddings table for the ANN index models#735
Constraint: Provide information about input list features to Merlin System [RMP] Establish a metadata standard for serializing information about Merlin components #489
Systems
DictArray
reference frommerlin.core.dispatch
core#163Column
abstraction systems#223Documentation
Examples
Small scale (see #352)
ColumnSchema
systems#173Blocker
The text was updated successfully, but these errors were encountered: