From 5933a183df3dc3896b5f7050b353d61a5f5cb01d Mon Sep 17 00:00:00 2001
From: satyajandhyala <satya.k.jandhyala@gmail.com>
Date: Fri, 7 Jul 2023 09:55:55 -0700
Subject: [PATCH] [Web/JS] Added missing L1Reduce and L2Reduce oprator kernels.
 (#16580)

### Description
Add missing L1Reduce and L2Reduce operator kernels.



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
---
 js/web/test/suite-test-list.jsonc             | 32 +++++++++----------
 .../providers/js/js_execution_provider.cc     | 10 ++++++
 .../core/providers/js/operators/reduce.cc     | 10 ++++++
 3 files changed, 36 insertions(+), 16 deletions(-)

diff --git a/js/web/test/suite-test-list.jsonc b/js/web/test/suite-test-list.jsonc
index 42be0e135a..17f0b1870c 100644
--- a/js/web/test/suite-test-list.jsonc
+++ b/js/web/test/suite-test-list.jsonc
@@ -884,22 +884,22 @@
       // "test_range_int32_type_negative_delta",
       "test_reciprocal_example",
       "test_reciprocal",
-      // "test_reduce_l1_default_axes_keepdims_example",
-      // "test_reduce_l1_default_axes_keepdims_random",
-      // "test_reduce_l1_do_not_keepdims_example",
-      // "test_reduce_l1_do_not_keepdims_random",
-      // "test_reduce_l1_keep_dims_example",
-      // "test_reduce_l1_keep_dims_random",
-      // "test_reduce_l1_negative_axes_keep_dims_example",
-      // "test_reduce_l1_negative_axes_keep_dims_random",
-      // "test_reduce_l2_default_axes_keepdims_example",
-      // "test_reduce_l2_default_axes_keepdims_random",
-      // "test_reduce_l2_do_not_keepdims_example",
-      // "test_reduce_l2_do_not_keepdims_random",
-      // "test_reduce_l2_keep_dims_example",
-      // "test_reduce_l2_keep_dims_random",
-      // "test_reduce_l2_negative_axes_keep_dims_example",
-      // "test_reduce_l2_negative_axes_keep_dims_random",
+      "test_reduce_l1_default_axes_keepdims_example",
+      "test_reduce_l1_default_axes_keepdims_random",
+      "test_reduce_l1_do_not_keepdims_example",
+      "test_reduce_l1_do_not_keepdims_random",
+      "test_reduce_l1_keep_dims_example",
+      "test_reduce_l1_keep_dims_random",
+      "test_reduce_l1_negative_axes_keep_dims_example",
+      "test_reduce_l1_negative_axes_keep_dims_random",
+      "test_reduce_l2_default_axes_keepdims_example",
+      "test_reduce_l2_default_axes_keepdims_random",
+      "test_reduce_l2_do_not_keepdims_example",
+      "test_reduce_l2_do_not_keepdims_random",
+      "test_reduce_l2_keep_dims_example",
+      "test_reduce_l2_keep_dims_random",
+      "test_reduce_l2_negative_axes_keep_dims_example",
+      "test_reduce_l2_negative_axes_keep_dims_random",
       "test_reduce_log_sum_asc_axes",
       "test_reduce_log_sum_default",
       "test_reduce_log_sum_desc_axes",
diff --git a/onnxruntime/core/providers/js/js_execution_provider.cc b/onnxruntime/core/providers/js/js_execution_provider.cc
index 9d38176c13..b09ccc3ae6 100644
--- a/onnxruntime/core/providers/js/js_execution_provider.cc
+++ b/onnxruntime/core/providers/js/js_execution_provider.cc
@@ -348,6 +348,16 @@ std::unique_ptr<KernelRegistry> RegisterKernels() {
       BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 11, 12, float, ReduceSum)>,
       BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 13, float, ReduceSum)>,
 
+      BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 1, 10, float, ReduceL1)>,
+      BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 11, 12, float, ReduceL1)>,
+      BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 13, 17, float, ReduceL1)>,
+      BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 18, float, ReduceL1)>,
+
+      BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 1, 10, float, ReduceL2)>,
+      BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 11, 12, float, ReduceL2)>,
+      BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 13, 17, float, ReduceL2)>,
+      BuildKernelCreateInfo<ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 18, float, ReduceL2)>,
+
       BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 1, 10, float, ReduceLogSum)>,
       BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 11, 12, float, ReduceLogSum)>,
       BuildKernelCreateInfo<ONNX_OPERATOR_VERSIONED_TYPED_KERNEL_CLASS_NAME(kJsExecutionProvider, kOnnxDomain, 13, 17, float, ReduceLogSum)>,
diff --git a/onnxruntime/core/providers/js/operators/reduce.cc b/onnxruntime/core/providers/js/operators/reduce.cc
index b2b760e5b3..21854fccc3 100644
--- a/onnxruntime/core/providers/js/operators/reduce.cc
+++ b/onnxruntime/core/providers/js/operators/reduce.cc
@@ -60,6 +60,16 @@ REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceSum, 1, 10);
 REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceSum, 11, 12);
 REGISTER_REDUCE_ELEMENTWISE_KERNEL(ReduceSum, 13);
 
+REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceL1, 1, 10);
+REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceL1, 11, 12);
+REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceL1, 13, 17);
+REGISTER_REDUCE_ELEMENTWISE_KERNEL(ReduceL1, 18);
+
+REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceL2, 1, 10);
+REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceL2, 11, 12);
+REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceL2, 13, 17);
+REGISTER_REDUCE_ELEMENTWISE_KERNEL(ReduceL2, 18);
+
 REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceLogSum, 1, 10);
 REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceLogSum, 11, 12);
 REGISTER_REDUCE_ELEMENTWISE_VERSIONED_KERNEL(ReduceLogSum, 13, 17);