[Hackathon 7th No.55] BUG FIX paddle.nn.functional.fold -part (#70102)

DrRyanHuang · web-flow · commit a93c8810e726 · 2025-01-03T11:00:53.000+08:00
* Update unary.cc

* add unitest

* modify comments
diff --git a/paddle/phi/infermeta/unary.cc b/paddle/phi/infermeta/unary.cc
@@ -1832,20 +1832,20 @@ void FoldInferMeta(const MetaTensor& x,
                         "but received strides_height: %d strides_width: %d.",
                         strides[0],
                         strides[1]));
-  // check dilations
+  // check output size
   PADDLE_ENFORCE_GT(output_height,
-                    1,
+                    0,
                     common::errors::InvalidArgument(
-                        "The `output_height` should be greater than one, "
+                        "The `output_height` should be greater than zero, "
                         "but received output_height: %d .",
                         output_height));
   PADDLE_ENFORCE_GT(output_width,
-                    1,
+                    0,
                     common::errors::InvalidArgument(
-                        "The `output_width` should be greater than one, "
+                        "The `output_width` should be greater than zero, "
                         "but received output_width: %d .",
                         output_width));
-  // check output size
+  // check dilations
   PADDLE_ENFORCE_GT(
       dilation_height,
       0,
diff --git a/test/legacy_test/test_fold_op.py b/test/legacy_test/test_fold_op.py
@@ -164,6 +164,20 @@ def init_data(self):
         self.x = np.random.rand(*input_shape).astype(np.float64)
 
 
+class TestFoldshape1d(TestFoldOp):
+    def init_data(self):
+        self.batch_size = 8
+        self.input_channels = 3 * 3 * 3
+        self.length = 3
+        self.kernel_sizes = [1, 3]
+        self.strides = [1, 1]
+        self.paddings = [0, 0, 0, 0]
+        self.dilations = [1, 1]
+        self.output_sizes = [1, 5]
+        input_shape = [self.batch_size, self.input_channels, self.length]
+        self.x = np.random.rand(*input_shape).astype(np.float64)
+
+
 class TestFoldAPI(TestFoldOp):
     # This is for test on paddle.nn.Fold
 
diff --git a/tools/codestyle/clang-tidy.py b/tools/codestyle/clang-tidy.py
@@ -493,13 +493,15 @@ def main():
                 file=sys.stderr,
             )
             subprocess.check_output(
-                'pip install --no-cache clang-tidy=="15.0.2.1"', shell=True
+                'python3 -mpip install --no-cache clang-tidy=="15.0.2.1"',
+                shell=True,
             )
     except:
         print(
             "clang-tidy not found, attempting auto-install...", file=sys.stderr
         )
         subprocess.check_output(
-            'pip install --no-cache clang-tidy=="15.0.2.1"', shell=True
+            'python3 -mpip install --no-cache clang-tidy=="15.0.2.1"',
+            shell=True,
         )
     main()

Original file line number	Diff line number	Diff line change
`@@ -493,13 +493,15 @@ def main():`
`493`	`493`	`file=sys.stderr,`
`494`	`494`	`)`
`495`	`495`	`subprocess.check_output(`
`496`		`- 'pip install --no-cache clang-tidy=="15.0.2.1"', shell=True`
	`496`	`+ 'python3 -mpip install --no-cache clang-tidy=="15.0.2.1"',`
	`497`	`+ shell=True,`
`497`	`498`	`)`
`498`	`499`	`except:`
`499`	`500`	`print(`
`500`	`501`	`"clang-tidy not found, attempting auto-install...", file=sys.stderr`
`501`	`502`	`)`
`502`	`503`	`subprocess.check_output(`
`503`		`- 'pip install --no-cache clang-tidy=="15.0.2.1"', shell=True`
	`504`	`+ 'python3 -mpip install --no-cache clang-tidy=="15.0.2.1"',`
	`505`	`+ shell=True,`
`504`	`506`	`)`
`505`	`507`	`main()`