Closed crtrentz closed 4 years ago
@benjamin-gorman @hjmjohnson
@crtrentz @benjamin-gorman Very nice. Thank you for identifying the other issues related to this.
The next step would be to write unit tests related to the way these will be used BEFORE modifying any of the main source code.
thanks, the InterpolationOrder
class with BI_*
seems to suggest spatially 2D? I feel using SPLINE0 - SPLINE5
is more appropriate here. otherwise they look good.
thanks, the
InterpolationOrder
class withBI_*
seems to suggest spatially 2D? I feel usingSPLINE0 - SPLINE5
is more appropriate here. otherwise they look good.
We found those on skimage.transform.warp after being redirected from skimage.transform.resize.
thanks, the
InterpolationOrder
class withBI_*
seems to suggest spatially 2D? I feel usingSPLINE0 - SPLINE5
is more appropriate here. otherwise they look good.We found those on skimage.transform.warp after being redirected from skimage.transform.resize.
still not sure if it is correct... for your first link, a few lines after the interpolation order the output shape docstring also suggests 2D:
output_shape tuple (rows, cols), optional Shape of the output image generated. By default the shape of the input image is preserved. Note that, even for multi-band images, only rows and columns need to be specified."
but for the second link, it is ND:
Size of the generated output image (rows, cols[, …][, dim]). If dim is not provided, the number of channels is preserved. In case the number of input channels does not equal the number of output channels a n-dimensional interpolation is applied.
had a quick look at the source code of these, in the end they call scipy.ndimage.map_coordinates
Co-authored-by: Benjamin Gorman benjamin-gorman@uiowa.edu
Is your feature request related to a problem? Please describe.
Iteration of [#403] [#407]
Describe the solution you'd like
Docstring, Enumeration, Type Hint
mode [1]
mode [2]
mode [3]
mode [4]
mode [5]
padding_mode [6]
padding_mode [7]
average [8]
reduction [9]
reduction [10]
w_type [11]
norm_type [12]
acti_type [13]
channel_matching [14]
method [15]
Describe alternatives/additions you've considered
class Mode(str, Enum):
Union[Sequence[Mode], Sequence[str], Mode, str]