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

Model card ++ for ABP, Phishing, Root cause analysis #1014

Merged
merged 12 commits into from
Jun 30, 2023
Merged

Model card ++ for ABP, Phishing, Root cause analysis #1014

merged 12 commits into from
Jun 30, 2023

Conversation

gbatmaz
Copy link
Contributor

@gbatmaz gbatmaz commented Jun 29, 2023 •
edited
Loading

Description

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@gbatmaz gbatmaz added non-breaking Non-breaking change improvement Improvement to existing functionality 3 - Ready for Review Data Science labels Jun 29, 2023
@gbatmaz gbatmaz requested a review from hsin-c June 29, 2023 21:16
@gbatmaz gbatmaz requested a review from a team as a code owner June 29, 2023 21:16
@mdemoret-nv
Copy link
Contributor

@gbatmaz You are missing some copyright headers. Please run python ci/scripts/copyright.py --git-modified-only --update-current-year --insert and commit the changes to allow CI to pass.

hsin-c
hsin-c requested changes Jun 30, 2023
View reviewed changes
models/model-cards/abp-model-card.md Show resolved Hide resolved
models/model-cards/abp-model-card.md Show resolved Hide resolved
models/model-cards/abp-model-card.md Show resolved Hide resolved
models/model-cards/abp-model-card.md Show resolved Hide resolved
models/model-cards/abp-model-card.md Show resolved Hide resolved
models/model-cards/root-cause-analysis-model-card.md Outdated Show resolved Hide resolved
models/model-cards/root-cause-analysis-model-card.md Outdated Show resolved Hide resolved
models/model-cards/root-cause-analysis-model-card.md Outdated Show resolved Hide resolved
models/model-cards/root-cause-analysis-model-card.md Outdated Show resolved Hide resolved
models/model-cards/root-cause-analysis-model-card.md Show resolved Hide resolved
@hsin-c
Copy link
Contributor

hsin-c commented Jun 30, 2023

/merge

@rapids-bot rapids-bot bot merged commit 3c4b767 into nv-morpheus:branch-23.07 Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hsin-c hsin-c hsin-c approved these changes

Assignees
No one assigned
Labels
Data Science improvement Improvement to existing functionality non-breaking Non-breaking change
Projects
Morpheus Boards
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gbatmaz @mdemoret-nv @hsin-c