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

[GSProcessing] Custom Data Split disorder #976

Closed
jalencato opened this issue Aug 16, 2024 · 2 comments · Fixed by #1039
Closed

[GSProcessing] Custom Data Split disorder #976

jalencato opened this issue Aug 16, 2024 · 2 comments · Fixed by #1039
Assignees
@jalencato
Labels
bug Something isn't working gsprocessing For issues and PRs related the the GSProcessing library
Milestone
0.4 release

Comments

@jalencato
Copy link
Collaborator

If we store our custom split generated masks in multiple files, it will cause the final generated graph in a disorder. A temporary fix can be used to restrict it to 1. But we can try to optimize it for better scalability issue.
@classicsong classicsong added bug Something isn't working gsprocessing For issues and PRs related the the GSProcessing library labels Aug 22, 2024
@classicsong classicsong added this to the 0.4 release milestone Aug 22, 2024
@jalencato jalencato linked a pull request Oct 16, 2024 that will close this issue
[GSProcessing] PR2 - Custom Data Split for MT #1039
Merged
@jalencato
Copy link
Collaborator Author

Close the issue as we have already solved this via PR: #1039

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jalencato jalencato

Labels
bug Something isn't working gsprocessing For issues and PRs related the the GSProcessing library
Projects
None yet
Milestone
0.4 release
Development

Successfully merging a pull request may close this issue.

[GSProcessing] PR2 - Custom Data Split for MT jalencato/graphstorm
3 participants
@classicsong @jalencato and others