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

[Homo Optimization] Optimization on GSF #686

Merged
merged 26 commits into from
Dec 20, 2023

Conversation

jalencato
Copy link
Collaborator

@jalencato jalencato commented Dec 12, 2023

Issue #, if available:

Description of changes:

Allow users to start training/inferring job without specifying target_ntype/target_etype on homogeneous graph.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

classicsong and others added 9 commits December 11, 2023 13:53
…wslabs#681)

*Issue #, if available:*
As we chain train/inference with node id remapping in the train and
inference pipeline, both train/inference entries and remap entry should
be more robust to unknown arguments.

Related issue: awslabs#674 

*Description of changes:*
Update builtin train/inference entries to accept unknown arguments.


By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.

Co-authored-by: Xiang Song <[email protected]>
…mes (awslabs#682)

*Issue #, if available:*
When --column-names does not cover all builtin names, the remap will
crash

Related issue: awslabs#674 

*Description of changes:*


By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.

---------

Co-authored-by: Xiang Song <[email protected]>
@jalencato jalencato changed the base branch from main to homo-optmization December 15, 2023 21:42
@jalencato jalencato added the ready able to trigger the CI label Dec 15, 2023
@jalencato jalencato marked this pull request as ready for review December 18, 2023 20:16
@jalencato jalencato linked an issue Dec 18, 2023 that may be closed by this pull request
docs/source/configuration/configuration-run.rst Outdated Show resolved Hide resolved
docs/source/configuration/configuration-run.rst Outdated Show resolved Hide resolved
python/graphstorm/config/argument.py Outdated Show resolved Hide resolved
python/graphstorm/config/argument.py Outdated Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Outdated Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Outdated Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Outdated Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Outdated Show resolved Hide resolved
tests/unit-tests/test_dataloading.py Show resolved Hide resolved
docs/source/configuration/configuration-run.rst Outdated Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Outdated Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Outdated Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Outdated Show resolved Hide resolved
python/graphstorm/dataloading/dataset.py Outdated Show resolved Hide resolved
tests/unit-tests/data_utils.py Outdated Show resolved Hide resolved
tests/unit-tests/data_utils.py Outdated Show resolved Hide resolved
tests/unit-tests/data_utils.py Outdated Show resolved Hide resolved
@jalencato jalencato merged commit 411e220 into awslabs:homo-optmization Dec 20, 2023
6 checks passed
classicsong added a commit that referenced this pull request Dec 20, 2023
*Issue #, if available:*

*Description of changes:*

Allow users to start training/inferring job without specifying
target_ntype/target_etype on homogeneous graph.

By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.

---------

Co-authored-by: xiang song(charlie.song) <[email protected]>
Co-authored-by: Xiang Song <[email protected]>
@jalencato jalencato deleted the homo_gsf branch February 2, 2024 17:37
@jalencato jalencato restored the homo_gsf branch February 2, 2024 17:37
@jalencato jalencato deleted the homo_gsf branch September 4, 2024 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready able to trigger the CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Native support for homogeneous graphs
2 participants