Skip to content

Conversation

@driazati
Copy link
Member

RoiAlign-16 introduces coordinate_transformation_mode, which should be set to 'output_half_pixel' to omit the pixel shift for the input (for a backward-compatible behavior). This PR should fix the failure in https://ci.tlcpack.ai/job/docker-images-ci/job/docker-image-run-tests/231/testReport/junit/cython.tests.python.frontend.onnx/test_forward/Test___frontend__GPU_3_of_6___test_roi_align_cuda_/

Moved #12869 to an in-repo branch

@sfvaroglu
Copy link
Contributor

Please review @driazati, thanks!

@Mousius Mousius merged commit 7dbc68d into main Sep 27, 2022
@Mousius Mousius deleted the sevin/test_roi_align_fix branch September 27, 2022 13:26
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
RoiAlign-16 introduces coordinate_transformation_mode, which should be set to 'output_half_pixel' to omit the pixel shift for the input (for a backward-compatible behavior). This PR should fix the failure in https://ci.tlcpack.ai/job/docker-images-ci/job/docker-image-run-tests/231/testReport/junit/cython.tests.python.frontend.onnx/test_forward/Test___frontend__GPU_3_of_6___test_roi_align_cuda_/

Co-authored-by: Sevin F. Varoglu <sfvaroglu@octoml.ai>
Co-authored-by: driazati <driazati@users.noreply.github.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.

3 participants