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

Using Select and Connect modules in pooling layers #7625

Merged
merged 7 commits into from
Jun 22, 2023
Merged

Conversation

puririshi98
Copy link
Contributor

@puririshi98 puririshi98 commented Jun 21, 2023

need to fix failing CI

@puririshi98 puririshi98 requested a review from rusty1s June 21, 2023 20:14
@puririshi98 puririshi98 self-assigned this Jun 21, 2023
@github-actions github-actions bot added the nn label Jun 21, 2023
@puririshi98 puririshi98 changed the title using select and connect modules Draft: using select and connect modules Jun 21, 2023
@codecov
Copy link

codecov bot commented Jun 22, 2023

Codecov Report

Merging #7625 (6ee296c) into master (0f31546) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #7625   +/-   ##
=======================================
  Coverage   91.59%   91.59%           
=======================================
  Files         452      452           
  Lines       25384    25383    -1     
=======================================
  Hits        23250    23250           
+ Misses       2134     2133    -1     
Impacted Files Coverage Δ
torch_geometric/nn/pool/asap.py 90.66% <100.00%> (+0.25%) ⬆️
torch_geometric/nn/pool/pan_pool.py 97.82% <100.00%> (+2.08%) ⬆️
torch_geometric/nn/pool/sag_pool.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@rusty1s rusty1s changed the title Draft: using select and connect modules Using Select and Connect modules in pooling layers Jun 22, 2023
@rusty1s rusty1s merged commit ac4c804 into master Jun 22, 2023
14 checks passed
@rusty1s rusty1s deleted the pooling-upt branch June 22, 2023 13:24
@puririshi98 puririshi98 mentioned this pull request Jun 22, 2023
4 tasks
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.

None yet

2 participants