-
Notifications
You must be signed in to change notification settings - Fork 182
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
CI: remove SVM+neighbors GPU deselections #1991
CI: remove SVM+neighbors GPU deselections #1991
Conversation
/intelci: run |
/intelci: run |
/intelci: run |
/intelci: run |
1 similar comment
/intelci: run |
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.
Crazy number of reactivations, CI looks good. Great job!
Private CI failures unrelated to reactivated tests. |
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.
Amazing!
It would be good for the future to have a mechanism that would show how many tests were enabled within a certain fix.
As an idea, for this we can simply run tests without deselected and compare the crashes with the deselection list. It seems that it is not difficult to even automate
Description
Unclear what changes made these no longer an issue, but looks like there are no device issues on svm and neighbors conformance tests. Adds 750+ GPU conformance tests back into validation.
Checklist to comply with before moving PR from draft:
PR completeness and readability
Testing
Performance