remove redundant assert and check

oneflow/python/nn/modules/constantpad2d.py

@@ -100,37 +100,25 @@ def __init__(self, padding: Union[int, tuple, list], value: Union[int, float] =
     def forward(self, x):
         _, _, h, w = x.shape
 
-        if (
-            self.padding[2] < h
-            and self.padding[3] < h
-            and self.padding[0] < w
-            and self.padding[1] < w
-        ):
-
-            if x.dtype in [flow.float32, flow.float16, flow.float64]:
-                floating_value = float(self.value)
-                integral_value = int(0)
-            else:
-                floating_value = float(0)
-                integral_value = int(self.value)
-
-            self._op = (
-                flow.builtin_op("constant_pad2d")
-                .Input("x")
-                .Output("y")
-                .Attr("padding", self.padding)
-                .Attr("floating_value", floating_value)
-                .Attr("integral_value", integral_value)
-                .Build()
-            )
-
-            res = self._op(x)[0]
-            return res
-
+        if x.dtype in [flow.float32, flow.float16, flow.float64]:
+            floating_value = float(self.value)
+            integral_value = int(0)
         else:
-            raise AssertionError(
-                "Padding size should be less than the corresponding input dimension. Please check."
-            )
+            floating_value = float(0)
+            integral_value = int(self.value)
+
+        self._op = (
+            flow.builtin_op("constant_pad2d")
+            .Input("x")
+            .Output("y")
+            .Attr("padding", self.padding)
+            .Attr("floating_value", floating_value)
+            .Attr("integral_value", integral_value)
+            .Build()
+        )
+
+        res = self._op(x)[0]
+        return res
 
 
 if __name__ == "__main__":

oneflow/python/nn/modules/padding.py

@@ -98,8 +98,6 @@ def __init__(self, padding: Union[int, tuple, list]):
             .Input("x")
             .Output("y")
             .Attr("padding", self.padding)
-            .Attr("floating_value", float(1.0))
-            .Attr("integral_value", int(0))
             .Build()
         )
 
@@ -183,8 +181,6 @@ def __init__(self, padding: Union[int, tuple]) -> None:
             .Input("x")
             .Output("y")
             .Attr("padding", boundary)
-            .Attr("floating_value", float(1.0))
-            .Attr("integral_value", int(0))
             .Build()
         )
 

oneflow/python/ops/pad.py

@@ -372,16 +372,8 @@ def pad_Job(x: tp.Numpy.Placeholder((1, 2, 3, 3))
         assert len(padding) == len(x.shape), ValueError(
             "padding boundry must be the same size of input dims"
         )
-        assert (
-            padding[2] < H and padding[3] < H and padding[0] < W and padding[1] < W
-        ), ValueError(
-            "Padding size should be less than the corresponding input dimension!"
-        )
         boundry = [padding[0], padding[1], padding[2], padding[3]]
     elif isinstance(padding, int):
-        assert padding < H and padding < W, ValueError(
-            "Padding size should be less than the corresponding input dimension!"
-        )
         boundry = [padding, padding, padding, padding]
     else:
         raise ValueError("padding must be in or list or tuple!")
@@ -456,16 +448,8 @@ def pad_Job(x: tp.Numpy.Placeholder((1, 2, 3, 3), const_value)
         assert len(padding) == len(x.shape), ValueError(
             "padding boundry must be the same size of input dims"
         )
-        assert (
-            padding[2] < H and padding[3] < H and padding[0] < W and padding[1] < W
-        ), ValueError(
-            "Padding size should be less than the corresponding input dimension!"
-        )
         boundry = [padding[0], padding[1], padding[2], padding[3]]
     elif isinstance(padding, int):
-        assert padding < H and padding < W, ValueError(
-            "Padding size should be less than the corresponding input dimension!"
-        )
         boundry = [padding, padding, padding, padding]
     else:
         raise ValueError("padding must be in or list or tuple!")