Skip to content

Conversation

@Deivanayaki-S
Copy link
Contributor

@Deivanayaki-S Deivanayaki-S commented May 5, 2025

This PR adds support for max_pool1d and max_pool3d operations, extending the existing max_pool2d functionality to handle 1D and 3D pooling. It includes the necessary mappings, and comprehensive tests for these new operations. Additionally, maxpool2d was refactored to match the result of pytorch implementation. By adding this support, the 3DResNet torchvision model is now able to run successfully.

@Deivanayaki-S
Copy link
Contributor Author

@tlopex Could you pls review this pr?

@Deivanayaki-S Deivanayaki-S marked this pull request as ready for review May 7, 2025 08:33
@tlopex tlopex self-assigned this May 8, 2025
Copy link
Member

@tlopex tlopex left a comment

Choose a reason for hiding this comment

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

LGTM!

@tlopex tlopex merged commit 64cea4a into apache:main May 8, 2025
12 checks passed
ShiboXing pushed a commit to ShiboXing/tvm that referenced this pull request Aug 10, 2025
…m and FX graph (apache#17919)

* add max_pool 1d and 3d op support and refactor 2d op

* update the layout used in max pool 1d op

* add mappings into fx translator and fix lint issues

* fix missing incorrect mappings and add module func

* update output tensor struct info for maxpool1d

* add docs for handling edge cases

---------

Co-authored-by: deivanayakisankaralingam <deiva@Deivanayaki>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants