-
Notifications
You must be signed in to change notification settings - Fork 157
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
Training/Inference Module DFP Production #669
Training/Inference Module DFP Production #669
Conversation
bsuryadevara
commented
Feb 6, 2023
•
edited
Loading
edited
- Added a file to derive DFP command line arguments.
- Created partial (some stages are implemented in cpp) inference pipeline module
- Created training pipeline module
- Added pipeline executable files to run the inference/training pipeline modules
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a couple comments; for the most part everything looks good. Just want to polish the presentation a bit.
examples/digital_fingerprinting/production/morpheus/dfp_duo_modules_inference.py
Outdated
Show resolved
Hide resolved
examples/digital_fingerprinting/production/morpheus/dfp_duo_modules_inference.py
Outdated
Show resolved
Hide resolved
examples/digital_fingerprinting/production/morpheus/dfp_duo_modules_inference.py
Outdated
Show resolved
Hide resolved
examples/digital_fingerprinting/production/morpheus/dfp_azure_modules_inference.py
Outdated
Show resolved
Hide resolved
examples/digital_fingerprinting/production/morpheus/dfp_azure_modules_inference.py
Outdated
Show resolved
Hide resolved
examples/digital_fingerprinting/production/morpheus/dfp_azure_modules_inference.py
Outdated
Show resolved
Hide resolved
examples/digital_fingerprinting/production/morpheus/dfp_azure_modules_inference.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just change encoding from 'latin1' to utf-8, and double check pipelines run correctly.
/merge |
- Added a file to derive DFP command line arguments. - Created partial (some stages are implemented in cpp) inference pipeline module - Created training pipeline module - Added pipeline executable files to run the inference/training pipeline modules Authors: - Bhargav Suryadevara (https://github.com/bsuryadevara) Approvers: - Devin Robison (https://github.com/drobison00) URL: nv-morpheus#669