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.

Sign up for GitHub

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

Jump to bottom

[ONNX] Fix circular padding to support dynamic axes #95647

Closed
wants to merge 2 commits into from
Closed

[ONNX] Fix circular padding to support dynamic axes #95647

wants to merge 2 commits into from

Commits on Mar 8, 2023

  1. [ONNX] Fix circular padding to support dynamic axes 

    This commit fixes a bug where the ONNX exporter for circular padding
queried the input tensor shape in order to get the correct 'end' index
for a slice node. This doesn't work when the axis in question is
has dynamic size. The commit fixes this by setting the 'end' index to INT_MAX,
which is the recommended way of slicing to the end of a dimension
with unknown size per ONNX spec.
    @ilyasher @pytorchmergebot
    ilyasher authored and pytorchmergebot committed Mar 8, 2023
    Configuration menu
    Copy the full SHA
    32e2414 View commit details
    Browse the repository at this point in the history

  2. Fix lint 

    Co-authored-by: Bowen Bao <bowbao@microsoft.com>
    @ilyasher @BowenBao @pytorchmergebot
    2 people authored and pytorchmergebot committed Mar 8, 2023
    Configuration menu
    Copy the full SHA
    f877b1d View commit details
    Browse the repository at this point in the history