-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Confidence Score to IPEntity files #9191
Conversation
Required items, please complete Change(s): - Modified all IPEntity files to determine Alert Priority based on Confidence Score - Added logic to filter rows where confidence score was less than 51 - Added alertDetailsOverride to coincide with the Alert Priority Reason for Change(s): - To add the Alert Priority Version Updated: - Yes Testing Completed: - Yes Checked that the validations are passing and have addressed any issues that are present: - No
Hello @mmelndezlujn, Thanks for raising this PR. |
Hello @mmelndezlujn please repackage the solution and please clear the validation fails. |
Hello @mmelndezlujn, Please repackage the solution. |
Hello @mmelndezlujn, Please repackage the solution after committing your final changes. |
Hello @mmelndezlujn, We are waiting for your response. |
Hello @mmelndezlujn, please respond to the above comments. |
Hello @mmelndezlujn please repackage the solution. |
Hello Manuel, Please work on the above requested changes. |
Hello there, @mmelndezlujn. Please notify us if any changes are still pending on your end. |
Hello there, @mmelndezlujn. Could you please respond to the remarks above? |
Hello! I don't think there are any pending changes. |
Ok Thanks for confirming can you please repackage the solution. |
Hello @mmelndezlujn, can you please work on the requested changes. |
Task pending on me to review this PR by @mmelndezlujn and help with packaging effort. Would get this addressed by 11/15. |
Thanks, Noted @aprakash13. |
Required items, please complete
Change(s):
Reason for Change(s):
Version Updated:
Testing Completed:
Checked that the validations are passing and have addressed any issues that are present: