Skip to content

Update builtin_op_importers.cpp#628

Open
QMassoz wants to merge 1 commit intoonnx:mainfrom
QMassoz:patch-1
Open

Update builtin_op_importers.cpp#628
QMassoz wants to merge 1 commit intoonnx:mainfrom
QMassoz:patch-1

Conversation

@QMassoz
Copy link
Copy Markdown

@QMassoz QMassoz commented Jan 18, 2021

Pytorch exports linear interpolation with align_corners=True using coordinate_transformation_mode = "align_corners" and not "asymmetric".

See https://github.com/pytorch/pytorch/blob/master/torch/onnx/symbolic_opset11.py line 214-215:
coordinate_transformation_mode = "asymmetric" if interpolate_mode == "nearest"
else "align_corners" if align_corners else "pytorch_half_pixel"

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 18, 2021

CLA assistant check
All committers have signed the CLA.

Pytorch exports linear interpolation with align_corners=True with coordinate_transformation_mode = "align_corners" and not "asymmetric".

See https://github.com/pytorch/pytorch/blob/master/torch/onnx/symbolic_opset11.py line 214-215:
coordinate_transformation_mode = "asymmetric" if interpolate_mode == "nearest" \
            else "align_corners" if align_corners else "pytorch_half_pixel"

Signed-off-by: QMassoz <quentin.massoz@gmail.com>
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