-
Notifications
You must be signed in to change notification settings - Fork 147
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
ABP nvsmi sample data generation #1108
ABP nvsmi sample data generation #1108
Conversation
The dataset generated using the script contains 18 of the 29 columns used to train model. Therefore, example inference pipeline will fail because of the missing columns. These are the 18 columns:
Tried retraining model with just the 18 columns using: Accuracy was still 100%. Deployed new model to Triton. Ran the inference pipeline against the new model using the dataset generated from the script. Pipeline ran all the way through with no errors. @gbatmaz would it be possible to use model trained on the 18 columns instead so that datasets from both |
@efajardo-nv I believe that this isnt specific to NetQ, but is more specific to the GPU you are using. Have you run the script on a different GPU (quadro vs geforce vs tesla). Some may not have the necessary means of monitoring |
@mdemoret-nv Yes, that's correct. The columns generated from the script are slightly different between Tesla V100 (126 columns) and Quadro RTX 8000 (124 columns) but both have same 18 columns that overlap with: |
Yes, it should be okay for them to modify and use whatever is available for the notebook since you've done the sanity check. For production, it's probably a better idea to retrain with their own data to make sure correct distinction can be done with their type of workloads. |
@efajardo-nv Is this still a draft? |
@mdemoret-nv i need to retrain the model using the 18 columns in order for pipeline to work using inference data generated from the script. should be okay according to @gbatmaz. i can do that now unless you see any issues with that. |
…into abp-nvsmi-data-gen
…into abp-nvsmi-data-gen
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
/ok to test |
/ok to test |
/ok to test |
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.
LGTM
/merge |
Description
Closes #1097
Checklist
[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.