-
Notifications
You must be signed in to change notification settings - Fork 311
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
Disable WCC test until we get get on an A100 to debug on #1870
Disable WCC test until we get get on an A100 to debug on #1870
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
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.
Approving ops-codeowner
file changes
Codecov Report
@@ Coverage Diff @@
## branch-21.12 #1870 +/- ##
===============================================
Coverage ? 70.11%
===============================================
Files ? 143
Lines ? 8827
Branches ? 0
===============================================
Hits ? 6189
Misses ? 2638
Partials ? 0 Continue to review full report at Codecov.
|
…e part of it calls weak by default and still crashes on Ampere.
@gpucibot merge |
#1870 disabled some tests to allow us to push some things through CI. While investigating this issue we discovered that we are having problem compiling the code in a container that uses the delivered 11.0 thrust. This PR is a different temporary workaround. We force downloading the desired version of thrust to be used as part of the cugraph build. The PR also reactivates all of the WCC testing. Authors: - Chuck Hastings (https://github.com/ChuckHastings) Approvers: - Brad Rees (https://github.com/BradReesWork) - Robert Maynard (https://github.com/robertmaynard) URL: #1883
CI testing on A100 is detecting a segmentation fault in the WCC implementation. We currently can't get developer access to an A100 in order to debug this. This PR will disable the WCC test. Once we can resolve the A100 issue we can enable this test again.