Skip to content

Commit dac6e76

Browse files
committed
Update grid_sample_test.cc to run on all execution providers & change GridSample NHWC version to 16.
1 parent e88f1b2 commit dac6e76

File tree

5 files changed

+69
-101
lines changed

5 files changed

+69
-101
lines changed

onnxruntime/contrib_ops/cuda/cuda_contrib_kernels.cc

+2-2
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ class ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kMSDomain, 1
204204
#endif
205205

206206
#ifdef ENABLE_CUDA_NHWC_OPS
207-
class ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kMSInternalNHWCDomain, 20, float, GridSample);
207+
class ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kMSInternalNHWCDomain, 16, float, GridSample);
208208
#endif
209209

210210
template <>
@@ -413,7 +413,7 @@ Status RegisterCudaContribKernels(KernelRegistry& kernel_registry) {
413413
#endif
414414

415415
#ifdef ENABLE_CUDA_NHWC_OPS
416-
BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kMSInternalNHWCDomain, 20, float, GridSample)>,
416+
BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kMSInternalNHWCDomain, 16, float, GridSample)>,
417417
#endif
418418
};
419419

onnxruntime/contrib_ops/cuda/grid_sample.cc

+6-2
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ namespace cuda {
1919
(*KernelDefBuilder::Create()) \
2020
.TypeConstraint("T1", DataTypeImpl::GetTensorType<T>()) \
2121
.TypeConstraint("T2", DataTypeImpl::GetTensorType<T>()), \
22-
GridSample<T, LAYOUT>);
22+
onnxruntime::contrib::cuda::GridSample<T, LAYOUT>);
2323

2424
REGISTER_KERNEL_TYPED(float, 1, LAYOUT_NCHW, kMSDomain)
25-
REGISTER_KERNEL_TYPED(float, 20, LAYOUT_NHWC, kMSInternalNHWCDomain)
25+
REGISTER_KERNEL_TYPED(float, 16, LAYOUT_NHWC, kMSInternalNHWCDomain)
2626

2727
template <typename T, bool IsNHWC>
2828
GridSample<T, IsNHWC>::GridSample(const OpKernelInfo& info) : CudaKernel(info) {
@@ -92,4 +92,8 @@ Status GridSample<T, IsNHWC>::ComputeInternal(OpKernelContext* context) const {
9292
}
9393
} // namespace cuda
9494
} // namespace contrib
95+
96+
namespace cuda {
97+
REGISTER_KERNEL_TYPED(float, 16, LAYOUT_NCHW, kOnnxDomain)
98+
} // namespace cuda
9599
} // namespace onnxruntime

onnxruntime/core/providers/cuda/cuda_execution_provider.cc

+2
Original file line numberDiff line numberDiff line change
@@ -1256,6 +1256,7 @@ class ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain,
12561256
class ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 16, double, LessOrEqual);
12571257
class ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 16, MLFloat16, LessOrEqual);
12581258
class ONNX_OPERATOR_VERSIONED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 16, 17, ScatterElements);
1259+
class ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 16, float, GridSample);
12591260

12601261
// Opset 17
12611262
class ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 17, float, LayerNormalization);
@@ -2143,6 +2144,7 @@ static Status RegisterCudaKernels(KernelRegistry& kernel_registry) {
21432144
BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 16, double, LessOrEqual)>,
21442145
BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 16, MLFloat16, LessOrEqual)>,
21452146
BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 16, 17, ScatterElements)>,
2147+
BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 16, float, GridSample)>,
21462148

21472149
// Opset 17
21482150
BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kCudaExecutionProvider, kOnnxDomain, 17, float, LayerNormalization)>,

0 commit comments

Comments
 (0)