-
Notifications
You must be signed in to change notification settings - Fork 130
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
[EPIC]: Replace dfencoder with NVTabular #517
Comments
The following is a quick breakdown of process and necessary steps to update DFEncoder to utilize NVTabular. Updates to Morpheus coreWe should update the
Updates to
|
Moving tracking issue to next release |
Deprioritizing |
Is this a new feature, an improvement, or a change to existing functionality?
Improvement
How would you describe the priority of this feature request
High
Please provide a clear description of problem this feature solves
Morpheus currently relies on a fork of dfencoder, especially for Digital Fingerprinting. There are a number of issues with this, including:
This should be replaced with something that is more performant and that is GPU-aware.
Describe your ideal solution
The Merlin team created NVTabular, and it appears that this is a suitable replacement for dfencoder in our pipelines. The steps to get there include:
Tasks
dfencoder
into the Morpheus repo #753Describe any alternatives you have considered
No response
Additional context
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: