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.

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

[distGB] fix local variable 'sorted_idx' referenced before assignment in convert_partition.py #7830

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

CfromBU
Copy link
Collaborator

@CfromBU CfromBU commented Oct 17, 2024

The code will crashed in convert_partition.py::_process_partition_gb when sorted_etypes is False. In this pr, I add the code when condition is False.

@dgl-bot
Copy link
Collaborator

dgl-bot commented Oct 17, 2024

Commit ID: 7dc49df9747b349d01675d5bc9caf1e0002e7881

Build ID: 2

Status: ❌ CI test failed in Stage [Authentication].

Report path: link

Full logs path: link

@dgl-bot
Copy link
Collaborator

dgl-bot commented Oct 17, 2024

Commit ID: 628be0d30590801c865a7f2a58f6788903a997ba

Build ID: 1

Status: ❌ CI test failed in Stage [Authentication].

Report path: link

Full logs path: link

@CfromBU
Copy link
Collaborator Author

CfromBU commented Oct 18, 2024

this pr is related to issue#7816

@CfromBU CfromBU requested a review from Rhett-Ying October 29, 2024 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants