Skip to content

Fix MNMG Louvain tests on Pascal architecture#1322

Merged
rapids-bot[bot] merged 4 commits intorapidsai:branch-0.18from
ChuckHastings:fix_mnmg_pascal
Jan 6, 2021
Merged

Fix MNMG Louvain tests on Pascal architecture#1322
rapids-bot[bot] merged 4 commits intorapidsai:branch-0.18from
ChuckHastings:fix_mnmg_pascal

Conversation

@ChuckHastings
Copy link
Copy Markdown
Collaborator

MNMG Louvain uses a feature not supported on Pascal. This PR updates the python unit tests so that it expects an exception to be raised if running on a Pascal GPU.

@ChuckHastings ChuckHastings requested a review from a team as a code owner January 5, 2021 19:26
@BradReesWork BradReesWork added 6 - Okay to Auto-Merge bug Something isn't working non-breaking Non-breaking change labels Jan 5, 2021
Copy link
Copy Markdown
Member

@afender afender left a comment

Choose a reason for hiding this comment

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

👍

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jan 5, 2021

Codecov Report

Merging #1322 (db777e3) into branch-0.18 (d69cff7) will decrease coverage by 0.15%.
The diff coverage is 23.07%.

Impacted file tree graph

@@               Coverage Diff               @@
##           branch-0.18    #1322      +/-   ##
===============================================
- Coverage        60.21%   60.06%   -0.16%     
===============================================
  Files               67       67              
  Lines             3049     3060      +11     
===============================================
+ Hits              1836     1838       +2     
- Misses            1213     1222       +9     
Impacted Files Coverage Δ
python/cugraph/utilities/utils.py 66.92% <18.18%> (-4.63%) ⬇️
python/cugraph/dask/common/mg_utils.py 37.50% <50.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d69cff7...db777e3. Read the comment docs.

@ChuckHastings
Copy link
Copy Markdown
Collaborator Author

rerun tests

@rapids-bot rapids-bot Bot merged commit 499d956 into rapidsai:branch-0.18 Jan 6, 2021
@BradReesWork BradReesWork added this to the 0.18 milestone Jan 20, 2021
@ChuckHastings ChuckHastings deleted the fix_mnmg_pascal branch February 10, 2021 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants