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.

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

How to for embedding op with categorify from scratch #1827

Merged
merged 7 commits into from
Jun 8, 2023

Conversation

jperez999
Copy link
Contributor

@jperez999 jperez999 commented May 25, 2023

This PR adds a test that illustrates how to use pretrained embeddings and how to reuse a categorify operator to either categorify data to what an item map, or categorify an item map to data that was already fitted in a workflow. It shows how to take that all the way to the dataloader where we can use it to load in embeddings for new data presented.

Requires the following core pr NVIDIA-Merlin/core#337

@jperez999 jperez999 added the enhancement New feature or request label May 25, 2023
@jperez999 jperez999 added this to the Merlin 23.06 milestone May 25, 2023
@jperez999 jperez999 self-assigned this May 25, 2023
Copy link
Contributor

@karlhigley karlhigley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, but the PR could use a title that's legible to people outside the team (since the PR titles are now part of our auto-generated release notes)

@jperez999 jperez999 changed the title Embedding op fresh How to for embedding op with categorify from scratch May 26, 2023
@github-actions
Copy link

Documentation preview

https://nvidia-merlin.github.io/NVTabular/review/pr-1827

@jperez999 jperez999 merged commit 25151f7 into NVIDIA-Merlin:main Jun 8, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants