-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[GraphBolt] Allow using multiple processes for GraphBolt partition conversion #7497
Conversation
To trigger regression tests:
|
4d61a06
to
8fd0758
Compare
@Rhett-Ying updated to use Python libs for mp |
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.
please add dedicated test for this change into https://github.com/dmlc/dgl/blob/master/tests/distributed/test_partition.py. and As distributed tests are disabled in CI. please run the tests(all in test_partition at least) in your local.
deb2bfc
to
19fa0e2
Compare
2f525d3
to
d9f5013
Compare
@Rhett-Ying could you take another look? I've had to use |
please fix the lint check error |
a9f598a
to
271160b
Compare
Hi @Rhett-Ying CI passes now, let me know if there's any work left here. |
Description
Add option to use multi-processing to speed up conversion of DGL to GraphBolt partitions.
Checklist
Please feel free to remove inapplicable items for your PR.
Changes