-
Notifications
You must be signed in to change notification settings - Fork 615
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
Add lookup table operator #1251
Conversation
efe97d3
to
9003501
Compare
d9ecef5
to
052f67c
Compare
!build |
CI MESSAGE: [899100]: BUILD STARTED |
CI MESSAGE: [899100]: BUILD PASSED |
Btw, did you forget to add python tests? |
Signed-off-by: Joaquin Anton <janton@nvidia.com>
052f67c
to
a6528b5
Compare
Yes, and the GPU implementation. I pushed it now |
Signed-off-by: Joaquin Anton <janton@nvidia.com>
c32d382
to
2b31d21
Compare
!build |
CI MESSAGE: [902153]: BUILD STARTED |
CI MESSAGE: [902153]: BUILD PASSED |
Signed-off-by: Joaquin Anton <janton@nvidia.com>
3cbe892
to
058b1eb
Compare
Signed-off-by: Joaquin Anton <janton@nvidia.com>
058b1eb
to
c40a664
Compare
!build |
CI MESSAGE: [903729]: BUILD STARTED |
CI MESSAGE: [903729]: BUILD PASSED |
Signed-off-by: Joaquin Anton <janton@nvidia.com> Signed-off-by: Jianjun Liu <00liujj@163.com>
ops, only int supported for input. |
@LiuHao-THU - what is your use case and how you want it to work? |
I'm Reimplement bayesian personalized ranking(a recommendation algorithm), the network is very small, however, I have over 200000 users in my datasets. Now, I'm using PyTorch data_loader, """ """
""" |
@LiuHao-THU - the lookup was rather designed for signal processing, than for tabular data. Have you tried to check RAPIDS? |
Signed-off-by: Joaquin Anton janton@nvidia.com
Why we need this PR?
Need a lookup operator, e.g. lookup weights for label values
What happened in this PR?
JIRA TASK: [DALI-1050]