-
Notifications
You must be signed in to change notification settings - Fork 12
Setup e2e integration test for graph store homogeneous inference #433
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
base: main
Are you sure you want to change the base?
Conversation
|
/e2e_test |
GiGL Automation@ 18:04:16UTC : 🔄 @ 18:24:09UTC : ❌ Workflow failed. |
|
/e2e_test |
GiGL Automation@ 18:25:14UTC : 🔄 |
|
/e2e_test |
GiGL Automation@ 21:17:12UTC : 🔄 |
|
/e2e_test |
GiGL Automation@ 21:27:12UTC : 🔄 @ 22:21:44UTC : ❌ Workflow failed. |
|
/e2e_test |
GiGL Automation@ 23:48:00UTC : 🔄 @ 24:51:24UTC : ✅ Workflow completed successfully. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please ignore, being reviewed in #432
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please ignore, being reviewed in #432
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please ignore, being reviewed in #432
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please ignore, being reviewed in #432
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please ignore, being reviewed in #432
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please ignore, being reviewed in #432
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please ignore, being reviewed in #432
|
/e2e_test |
GiGL Automation@ 01:37:59UTC : 🔄 @ 03:05:51UTC : ✅ Workflow completed successfully. |
Scope of work done
Please focus your reviews on the code under
examples/link_prediction/graph_store. Everything else should be reviewed in other prs (mainly #432 )Specifically,
examples/link_prediction/graph_store/homogeneous_inference.pyis the most important file here.As a follow up to this I will make it so users can customize the server code.
The main differences between this code and the "colocated" mode (which should be documented in the docstring) are:
init_compute_nodeto setup the entire heterogeneous training clusterGraphStoreInfoto capture the cluster topologyRemoteDistDatasetinstead ofDistDatasetas a pointer to theDistDatasetstored on the storage cluster.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