-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Fix dimension in edge filter selection #4629
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4629 +/- ##
=======================================
Coverage 82.88% 82.88%
=======================================
Files 316 316
Lines 16677 16678 +1
=======================================
+ Hits 13823 13824 +1
Misses 2854 2854
Continue to review full report at Codecov.
|
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.
We should fix this for node attributes as well, right?
Agreed, I'll do that. Edit: oh I see you did it for me, thanks! |
Aims to fix an issue reported in #4026. The filtering of attributes that have the same shape as
edge_index
are not handled correctly. This PR tries to fix that behaviour