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

Enable temporarily disabled MG tests #3837

Merged
merged 20 commits into from
Sep 26, 2023
Merged

Enable temporarily disabled MG tests #3837

merged 20 commits into from
Sep 26, 2023

Conversation

naimnv
Copy link
Contributor

@naimnv naimnv commented Aug 30, 2023 •
edited
Loading

Enable TEMPORARILY disable single-GPU "MG" tests
And Skip deleting copied Dataframe while creating distributed graph from cudf edge-lists.
Ideally we would like to merger this PR once the issue 3790 is closed, but me might need to merger it if the issue is not resolved before the next release.

@naimnv naimnv requested a review from a team as a code owner August 30, 2023 18:10
@naimnv naimnv added the DO NOT MERGE Hold off on merging; see PR for details label Aug 30, 2023
@copy-pr-bot
Copy link

copy-pr-bot bot commented Aug 30, 2023

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@naimnv naimnv self-assigned this Aug 30, 2023
@naimnv naimnv added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Aug 30, 2023
@BradReesWork
Copy link
Member

/ok to test

ajschmidt8
ajschmidt8 approved these changes Sep 1, 2023
View reviewed changes
Copy link
Member

@ajschmidt8 ajschmidt8 left a 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

@naimnv naimnv requested a review from a team as a code owner September 20, 2023 23:05
@ChuckHastings
Copy link
Collaborator

/ok to test

@rlratzel rlratzel removed the DO NOT MERGE Hold off on merging; see PR for details label Sep 21, 2023
@naimnv
Copy link
Contributor Author

naimnv commented Sep 21, 2023

/ok to test

@BradReesWork
Copy link
Member

/okay to test

@rlratzel rlratzel mentioned this pull request Sep 22, 2023
Closed
@naimnv
Copy link
Contributor Author

naimnv commented Sep 22, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Sep 22, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Sep 22, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Sep 25, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Sep 25, 2023

/ok to test

@BradReesWork BradReesWork added this to the 23.10 milestone Sep 25, 2023
@BradReesWork
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 8b02e24 into rapidsai:branch-23.10 Sep 26, 2023
56 of 57 checks passed
@rlratzel rlratzel mentioned this pull request Sep 27, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajschmidt8 ajschmidt8 ajschmidt8 approved these changes

@rlratzel rlratzel rlratzel approved these changes

Assignees

@naimnv naimnv

Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Milestone
23.10
Development

Successfully merging this pull request may close these issues.
5 participants
@naimnv @BradReesWork @ChuckHastings @rlratzel @ajschmidt8