Skip to content

Enable max label per anchor in data splitter#589

Open
yliu2-sc wants to merge 6 commits intomainfrom
yliu2/max_label_per_anchor
Open

Enable max label per anchor in data splitter#589
yliu2-sc wants to merge 6 commits intomainfrom
yliu2/max_label_per_anchor

Conversation

@yliu2-sc
Copy link
Copy Markdown
Collaborator

Scope of work done

In previous experiments, for non user defined labels where labels are generated from 1 hop, we need to enable limiting num max labels per anchor to reduce blow up in memory.

This PR updates to include the option through configs to limit max mum labels per anchor.

Where is the documentation for this feature?: N/A

Did you add automated tests or write a test plan?

Updated Changelog.md? NO

Ready for code review?: NO

Comment thread gigl/utils/data_splitters.py Outdated
Comment thread gigl/utils/data_splitters.py Outdated
Comment thread gigl/distributed/graph_store/storage_utils.py Outdated
Comment thread tests/unit/utils/data_splitters_test.py Outdated
Copy link
Copy Markdown
Collaborator

@kmontemayor2-sc kmontemayor2-sc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Yozen!

Comment thread tests/unit/utils/data_splitters_test.py Outdated
Comment thread gigl/distributed/graph_store/storage_utils.py
Copy link
Copy Markdown
Collaborator

@mkolodner-sc mkolodner-sc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stamp

@yliu2-sc yliu2-sc added this pull request to the merge queue Apr 30, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 30, 2026
@yliu2-sc yliu2-sc enabled auto-merge May 1, 2026 17:39
@yliu2-sc yliu2-sc added this pull request to the merge queue May 1, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 1, 2026
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.

3 participants