Skip to content

Conversation

@shingjan
Copy link

@shingjan shingjan commented Jun 27, 2022

This PR intends to add aten::pad in the pytorch frontend and refactor the padding-related ops.

cc: @masahi

"aten::replication_pad1d": self.make_pad("edge"),
"aten::replication_pad2d": self.make_pad("edge"),
"aten::replication_pad3d": self.make_pad("edge"),
"aten::pad": self.pad,
Copy link
Author

Choose a reason for hiding this comment

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

Seems to me that only with this change test_forward_functional_pad was able to work on my local end, is this test disabled before on CI? This is also the reason why I am not adding a new test case. @masahi

Copy link
Member

Choose a reason for hiding this comment

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

They should be tested. There are tests like test_forward_constant_pad1d and test_forward_reflection_pad1d() in pytorch/test_forward.py.

Can you test on PT 1.10? This is the version we use on CI. From our earlier conversation, I'm guessing that aten::pad is not in PT 1.10, so we cannot test his on CI.

YJ Shi added 2 commits June 27, 2022 12:02
@masahi masahi merged commit 993f728 into apache:main Jun 28, 2022
@shingjan shingjan deleted the torch_pad branch July 5, 2022 22:58
blackkker pushed a commit to blackkker/tvm that referenced this pull request Jul 7, 2022
* add aten::pad

* fix

* fix CI
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
* add aten::pad

* fix

* fix CI
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