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

Move ai explainability suite in dedicated directory #1575

Merged
merged 5 commits into from
Jul 1, 2024
Merged

Move ai explainability suite in dedicated directory #1575

merged 5 commits into from
Jul 1, 2024

Conversation

bdattoma
Copy link
Contributor

@bdattoma bdattoma commented Jun 28, 2024 •
edited
Loading

PR validation:

  • dryrun clean: PASS
  • execute bias metrics suite: rhods-ci-pr-test/3062 the failures are due to broken tests after changes in the product.

NOTE: includes a small fix even if out of scope for the PR 5c2fa4e

@bdattoma bdattoma added needs testing Needs to be tested in Jenkins misc Miscelaneus (PR will be listed in release-notes) labels Jun 28, 2024
@bdattoma bdattoma self-assigned this Jun 28, 2024
@github-actions GitHub Actions
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
496 0 0 496 100

@bdattoma bdattoma added verified This PR has been tested with Jenkins and removed needs testing Needs to be tested in Jenkins labels Jun 28, 2024
nehachopra27
nehachopra27 approved these changes Jun 28, 2024
View reviewed changes
Copy link
Contributor

@nehachopra27 nehachopra27 left a comment

Choose a reason for hiding this comment

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

LGTM

@bdattoma bdattoma merged commit 7d62006 into red-hat-data-services:master Jul 1, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nehachopra27 nehachopra27 nehachopra27 approved these changes

@mwaykole mwaykole mwaykole approved these changes

@jgarciao jgarciao Awaiting requested review from jgarciao

@jiridanek jiridanek Awaiting requested review from jiridanek

@jstourac jstourac Awaiting requested review from jstourac

@tarukumar tarukumar Awaiting requested review from tarukumar

Assignees

@bdattoma bdattoma

Labels
misc Miscelaneus (PR will be listed in release-notes) verified This PR has been tested with Jenkins
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bdattoma @nehachopra27 @mwaykole