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.

Sign up for GitHub

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

Jump to bottom

ransomware ds requirements #196

Merged
4 commits merged into from
Jun 28, 2022
Merged

ransomware ds requirements #196

4 commits merged into from
Jun 28, 2022

Conversation

raykallen
Copy link
Contributor

added training data and notebook. updated model info and readme.

closes #195

@raykallen
ransomware ds requirements
dd00cdb 
added training data and notebook. updated model info and readme.
@raykallen raykallen requested a review from a team as a code owner June 24, 2022 19:25
@raykallen raykallen added 4 - Waiting on Reviewer non-breaking Non-breaking change feature request New feature or request labels Jun 24, 2022
@raykallen
Update ransomware-appshield-training.ipynb
0fb1723 
update model eval in training
tzemicheal
tzemicheal approved these changes Jun 28, 2022
View reviewed changes
Copy link
Contributor

@tzemicheal tzemicheal left a comment

Choose a reason for hiding this comment

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

LGTM

@raykallen raykallen assigned raykallen and unassigned bsuryadevara Jun 28, 2022
bsuryadevara
bsuryadevara approved these changes Jun 28, 2022
View reviewed changes
Copy link
Contributor

@bsuryadevara bsuryadevara left a comment

Choose a reason for hiding this comment

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

LGTM

@raykallen
Copy link
Contributor Author

@gpucibot merge

@ghost ghost merged commit 30f275e into nv-morpheus:branch-22.06 Jun 28, 2022
@raykallen raykallen deleted the ransomware-ds-reqs branch July 5, 2022 19:59
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bsuryadevara bsuryadevara bsuryadevara approved these changes

@tzemicheal tzemicheal tzemicheal approved these changes

Assignees

@raykallen raykallen

Labels
feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@raykallen @bsuryadevara @tzemicheal