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.

Sign up for GitHub

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

Jump to bottom

Add operators for LightFM models #35

Open
karlhigley opened this issue Mar 14, 2022 · 3 comments
Open

Add operators for LightFM models #35

karlhigley opened this issue Mar 14, 2022 · 3 comments
Assignees
@benfred @oliverholworthy
Labels
enhancement New feature or request

Comments

@karlhigley
Copy link
Contributor

No description provided.

@karlhigley karlhigley changed the title Implicit and LightFM ensemble API (22.05) Add operators for Implicit and LightFM models Mar 14, 2022
@karlhigley karlhigley transferred this issue from NVIDIA-Merlin/Merlin Mar 14, 2022
@karlhigley karlhigley added this to the Merlin 22.05 milestone Mar 22, 2022
@karlhigley karlhigley added the enhancement New feature or request label Mar 22, 2022
@viswa-nvidia viswa-nvidia changed the title Add operators for Implicit and LightFM models Add operators for LightFM models Aug 8, 2022
@viswa-nvidia
Copy link

Changed the title to only lightfm models. There is a PR already for Implicit models

@viswa-nvidia viswa-nvidia mentioned this issue Aug 22, 2022
Closed
@karlhigley karlhigley modified the milestones: Merlin 22.08, Merlin 22.09 Aug 26, 2022
@EvenOldridge
Copy link
Member

@oliverholworthy correct me if I'm wrong but your PR for serving implicit also covers this does it not?

@oliverholworthy
Copy link
Member

@oliverholworthy correct me if I'm wrong but your PR for serving implicit also covers this does it not?

The operator for serving implicit is can only be used with implicit models. For LightFM support will need something different. So this issue has not been completed.

@rnyak rnyak mentioned this issue Sep 7, 2022
Open
@karlhigley karlhigley removed this from the Merlin 22.11 milestone Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benfred benfred

@oliverholworthy oliverholworthy

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@benfred @karlhigley @oliverholworthy @EvenOldridge @viswa-nvidia