Skip to content

Conversation

puririshi98
Copy link
Contributor

@puririshi98 puririshi98 commented Aug 12, 2025

Have noticed issues using this with our customers and it is no longer mantained by Intel. We should recommend using CuGraph which is only improving in terms of stability, popularity, and features.

Have noticed issues using this with our customers and it is no longer mantained by Intel. We should recommend using CuGraph which is only growing in popularity and contributions
@puririshi98 puririshi98 changed the title deprecate torch_geometric.distrbuted deprecate torch_geometric.distributed Aug 12, 2025
@puririshi98 puririshi98 changed the title deprecate torch_geometric.distributed deprecate torch_geometric.distributed Aug 12, 2025
@github-actions github-actions bot added the ci label Aug 12, 2025
@puririshi98
Copy link
Contributor Author

@rusty1s @akihironitta @wsad1 ready for re-review and merge

@akihironitta akihironitta self-assigned this Sep 4, 2025
@akihironitta akihironitta changed the title deprecate torch_geometric.distributed Deprecate torch_geometric.distributed Sep 5, 2025
@akihironitta akihironitta modified the milestones: 2.8.0, 2.7.0 Sep 5, 2025
Comment on lines +207 to +208
# TODO(rishipuri98): Remove usage of `torch_geometric.distributed` from `torch_geometric.llm`
"ignore:.*torch_geometric.distributed.*:DeprecationWarning",
Copy link
Member

Choose a reason for hiding this comment

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

@puririshi98 Mind taking care of this as a follow-up? Otherwise, it'll show the warning when user imports torch_geometric.llm.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes its on roadmap to replace the remaining usage of it with cugraph. WIP currently

@akihironitta akihironitta merged commit af00fcf into master Sep 20, 2025
16 checks passed
@akihironitta akihironitta deleted the deprecate-torch-geometric-distrbuted branch September 20, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants