Skip to content

Conversation

@tzuhanchang
Copy link
Contributor

Before the fix, tensors can only be concatenated over dim=0. The dim argument was not used by any operation in the function. This update allows the tensors to be concatenated over any given dimension.

@tzuhanchang tzuhanchang requested a review from wsad1 as a code owner November 6, 2024 22:49
@tzuhanchang tzuhanchang changed the title Fix torch_geometric.utils.group_cat concatenation dim Fix utils.group_cat concatenating dimension Nov 6, 2024
Copy link
Member

@rusty1s rusty1s left a comment

Choose a reason for hiding this comment

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

Well spotted, thanks!

@rusty1s rusty1s merged commit 599fce9 into pyg-team:master Nov 13, 2024
19 checks passed
mattjhayes3 pushed a commit to mattjhayes3/pytorch_geometric that referenced this pull request Dec 14, 2024
Before the fix, tensors can only be concatenated over `dim=0`. The `dim`
argument was not used by any operation in the function. This update
allows the tensors to be concatenated over any given dimension.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

2 participants