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

[DistGB] save as graphbolt graph directly after partition test case #7724

Merged
merged 22 commits into from
Sep 5, 2024
Merged

[DistGB] save as graphbolt graph directly after partition test case #7724

merged 22 commits into from
Sep 5, 2024

Conversation

CfromBU
Copy link
Collaborator

@CfromBU CfromBU commented Aug 20, 2024

modify the test_partiton.py

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 20, 2024

To trigger regression tests:

  • @dgl-bot run [instance-type] [which tests] [compare-with-branch];
    For example: @dgl-bot run g4dn.4xlarge all dmlc/master or @dgl-bot run c5.9xlarge kernel,api dmlc/master

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 20, 2024

Commit ID: 49747cd56b92054f1b17945902b1566817fbec66

Build ID: 1

Status: ❌ CI test failed in Stage [Lint Check].

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 20, 2024

Commit ID: 4ee04fa00a60184b05a5321afac54b1a86f334c9

Build ID: 2

Status: ❌ CI test failed in Stage [Lint Check].

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 20, 2024

Commit ID: a9df022f6574b1fc6b19487a5d769912d758510b

Build ID: 3

Status: ❌ CI test failed in Stage [Distributed Torch CPU Unit test].

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 21, 2024

Commit ID: 39d90c5f163c48feba94813244fccda0d2ed1e75

Build ID: 4

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

@CfromBU CfromBU requested a review from Rhett-Ying August 21, 2024 05:55
This was referenced Aug 21, 2024
Merged
Merged
@CfromBU
Copy link
Collaborator Author

CfromBU commented Aug 21, 2024

order:
#7724 partition test case
#7727 partition
#7728 dist partition
#7690 dist partition test case

@CfromBU CfromBU mentioned this pull request Aug 21, 2024
Open
Rhett-Ying
Rhett-Ying requested changes Aug 21, 2024
View reviewed changes
python/dgl/distributed/partition.py Outdated Show resolved Hide resolved
python/dgl/distributed/partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 23, 2024

Commit ID: 2ec778a3fcb7fc25c3d87f27f66d5ed9b3a17985

Build ID: 5

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

@CfromBU
Copy link
Collaborator Author

CfromBU commented Aug 23, 2024

I modified the function in test case that relevant to graphbolt, but didn't change the function about DGLGraph.

@CfromBU CfromBU requested a review from Rhett-Ying August 23, 2024 08:14
Rhett-Ying
Rhett-Ying requested changes Aug 26, 2024
View reviewed changes
python/dgl/distributed/partition.py Outdated Show resolved Hide resolved
python/dgl/distributed/partition.py Show resolved Hide resolved
python/dgl/distributed/partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 26, 2024

Commit ID: abc8c808f0ee4dbbb7692b54dcbd5ec764393fcb

Build ID: 6

Status: ❌ CI test failed in Stage [Lint Check].

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 26, 2024

Commit ID: 3ff5f7f2566d9419187d4a0d1eeef3086f9643e5

Build ID: 7

Status: ⚪️ CI test cancelled due to overrun.

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 26, 2024

Commit ID: 2c5e59919ac8a26b2c85fc73be53cc8acddb23c4

Build ID: 8

Status: ⚪️ CI test cancelled due to overrun.

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 26, 2024

Commit ID: b8f2fd44a6ff4c6b28cec7d4a90a21a13c979b5a

Build ID: 9

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Aug 28, 2024

Commit ID: 285e0725f7bcdc771f4485528bc833affffdf9b1

Build ID: 11

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

Rhett-Ying
Rhett-Ying reviewed Aug 28, 2024
View reviewed changes
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 1, 2024

Commit ID: 2080927e5fda1642b636b2962a770c4691a61921

Build ID: 12

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 2, 2024

Commit ID: 80aa7ebb70957e7a7784ba1789d64d8d770891db

Build ID: 13

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

@CfromBU CfromBU requested a review from Rhett-Ying September 2, 2024 03:49
@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 2, 2024

Commit ID: 590ae6b77467c36a2a67b10412e7f6762bd866a4

Build ID: 14

Status: ⚪️ CI test cancelled due to overrun.

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 2, 2024

Commit ID: f504cbb4feb38efc2b4ddcdf956ecf5a3e7e055c

Build ID: 15

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

Rhett-Ying
Rhett-Ying reviewed Sep 2, 2024
View reviewed changes
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 4, 2024

Commit ID: 4e65f093bf4c1327052871a6f8e2d598376a48eb

Build ID: 16

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

@Rhett-Ying Rhett-Ying self-requested a review September 4, 2024 12:32
@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 4, 2024

Commit ID: 8408d7267e66813765527bd46989cd07873d91ad

Build ID: 17

Status: ⚪️ CI test cancelled due to overrun.

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 4, 2024

Commit ID: 2be6fceebda4dede472f7f80eec33c5b580832dc

Build ID: 18

Status: ⚪️ CI test cancelled due to overrun.

Report path: link

Full logs path: link

Rhett-Ying
Rhett-Ying reviewed Sep 4, 2024
View reviewed changes
python/dgl/distributed/partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
tests/distributed/test_partition.py Show resolved Hide resolved
tests/distributed/test_partition.py Outdated Show resolved Hide resolved
@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 4, 2024

Commit ID: f5222232155ffae371d818c1ab13dca67fb134af

Build ID: 19

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 5, 2024

Commit ID: eb93312bfcf77425d894b29492934fb65d839820

Build ID: 20

Status: ⚪️ CI test cancelled due to overrun.

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Sep 5, 2024

Commit ID: 6f2b8d52cede9a75e41bbe000e6715cd6ef547cb

Build ID: 21

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link

@CfromBU CfromBU merged commit 0734e33 into dmlc:master Sep 5, 2024
2 checks passed
lijialin03 pushed a commit to lijialin03/dgl that referenced this pull request Jan 6, 2025
@CfromBU @lijialin03
[DistGB] save as graphbolt graph directly after partition test case (d…
6c2964c 
…mlc#7724)

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rhett-Ying Rhett-Ying Rhett-Ying approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@CfromBU @dgl-bot @Rhett-Ying