Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Fix inconsistent interpolation method values#16212

Merged
marcoabreu merged 1 commit intoapache:masterfrom
RayzhiYang:patch-1
Sep 22, 2019
Merged

Fix inconsistent interpolation method values#16212
marcoabreu merged 1 commit intoapache:masterfrom
RayzhiYang:patch-1

Conversation

@RayzhiYang
Copy link
Contributor

@RayzhiYang RayzhiYang commented Sep 19, 2019

In cv2, bicubic interpolation should be represented by 2, and area-based interpolation should be 3. This is correct in the docstring of imresize method, but wrong in the docstring of several other methods, which can be quite confusing when following the documentation.

In cv2, bicubic interpolation should be represented by 2, and area-based interpolation should be 3. This is correct in the docstring of `imresize` method, but wrong in the docstring of several other methods, which can be quite confusing when following the documentation.
@RayzhiYang RayzhiYang requested a review from szha as a code owner September 19, 2019 13:34
@marcoabreu marcoabreu merged commit 93b794e into apache:master Sep 22, 2019
drivanov pushed a commit to drivanov/incubator-mxnet that referenced this pull request Sep 26, 2019
In cv2, bicubic interpolation should be represented by 2, and area-based interpolation should be 3. This is correct in the docstring of `imresize` method, but wrong in the docstring of several other methods, which can be quite confusing when following the documentation.
larroy pushed a commit to larroy/mxnet that referenced this pull request Sep 28, 2019
In cv2, bicubic interpolation should be represented by 2, and area-based interpolation should be 3. This is correct in the docstring of `imresize` method, but wrong in the docstring of several other methods, which can be quite confusing when following the documentation.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants