Fix some typos (reviced, paramterless_type, etc.) (#61778)

co63oc · web-flow · commit dd0bb299e386 · 2024-02-22T10:56:40.000+08:00
diff --git a/paddle/fluid/operators/match_matrix_tensor_op.cc b/paddle/fluid/operators/match_matrix_tensor_op.cc
@@ -165,15 +165,15 @@ void MatchMatrixTensorOP::InferShape(framework::InferShapeContext* ctx) const {
         x_desc->GetLoDLevel(),
         1,
         platform::errors::InvalidArgument("The LoD level of Input(X) should be "
-                                          "greater than 1, but reviced %d.",
+                                          "greater than 1, but received %d.",
                                           x_desc->GetLoDLevel()));
     framework::VarDesc* y_desc =
         PADDLE_GET(framework::VarDesc*, ctx->GetInputVarPtrs("Y")[0]);
     PADDLE_ENFORCE_GE(
         y_desc->GetLoDLevel(),
         1,
         platform::errors::InvalidArgument("The LoD level of Input(Y) should be "
-                                          "greater than 1, but reviced %d.",
+                                          "greater than 1, but received %d.",
                                           y_desc->GetLoDLevel()));
     ctx->ShareLoD("X", "Out");
   }
diff --git a/test/cpp/new_executor/standalone_executor_pir_test.cc b/test/cpp/new_executor/standalone_executor_pir_test.cc
@@ -296,7 +296,7 @@ TEST(StandaloneExecutor, while_op) {
                      std::vector<int64_t>{1}, 10, phi::DataType::INT32)
                  .out();
 
-  // comput condition value: i <= ten
+  // compute condition value: i <= ten
   auto cond_value = builder.Build<LessEqualOp>(i, ten).out();
 
   auto while_op =
@@ -312,7 +312,7 @@ TEST(StandaloneExecutor, while_op) {
           .out();
   auto new_i = builder.Build<AddOp>(body_i_argument, one).out();
 
-  // comput new condition value: new_i <= new_ten
+  // compute new condition value: new_i <= new_ten
   auto new_cond_value =
       builder.Build<LessEqualOp>(new_i, body_ten_argument).out();
 
diff --git a/test/cpp/pir/control_flow_dialect/while_op_test.cc b/test/cpp/pir/control_flow_dialect/while_op_test.cc
@@ -53,7 +53,7 @@ TEST(while_op_test, base) {
       builder.Build<FullOp>(std::vector<int64_t>{1}, 10, phi::DataType::INT32)
           .out();
 
-  // comput condition value: i < ten
+  // compute condition value: i < ten
   auto cond_value = builder.Build<LessThanOp>(i, ten).out();
 
   auto while_op =
@@ -69,7 +69,7 @@ TEST(while_op_test, base) {
           .out();
   auto new_i = builder.Build<AddOp>(body_i_argument, one).out();
 
-  // comput new condition value: new_i < new_ten
+  // compute new condition value: new_i < new_ten
   auto new_cond_value =
       builder.Build<LessThanOp>(new_i, body_ten_argument).out();
 
@@ -124,7 +124,7 @@ TEST(while_op_test, network_with_backward) {
   auto new_i = builder.Build<AddOp>(body_i_argument, one).out();
   auto new_x = builder.Build<AddOp>(body_x_argument, y).out();
 
-  // comput new condition value: new_i < new_ten
+  // compute new condition value: new_i < new_ten
   auto new_cond_value = builder.Build<LessThanOp>(new_i, ten).out();
 
   builder.Build<pir::TuplePushOp>(
diff --git a/test/cpp/pir/core/ir_type_converter_test.cc b/test/cpp/pir/core/ir_type_converter_test.cc
@@ -51,7 +51,7 @@ void test_parameterless_type_helper() {
                                    (test_parameterless_type<IR_TYPE>(), 0)...};
 }
 
-TEST(TypeConverterTest, paramterless_type) {
+TEST(TypeConverterTest, parameterless_type) {
   test_parameterless_type_helper<pir::UInt8Type,
                                  pir::Int8Type,
                                  pir::BFloat16Type,
diff --git a/test/cpp/pir/core/type_interface_test.cc b/test/cpp/pir/core/type_interface_test.cc
@@ -23,7 +23,7 @@
 #include "test/cpp/pir/tools/test_dialect.h"
 #include "test/cpp/pir/tools/test_op.h"
 
-TEST(shapedtype_test, shapedtype_test) {
+TEST(shape_dtype_test, shape_dtype_test) {
   pir::IrContext *ctx = pir::IrContext::Instance();
   pir::Dialect *test_dialect = ctx->GetOrRegisterDialect<test::TestDialect>();
   ctx->GetOrRegisterDialect<paddle::dialect::OperatorDialect>();
diff --git a/test/cpp/pir/pattern_rewrite/drr_same_type_binding_test.cc b/test/cpp/pir/pattern_rewrite/drr_same_type_binding_test.cc
@@ -30,7 +30,7 @@
                                     /  |  \  \  \
                                   /    |   \   \    \
              full               /      |    |    \     \           full_tmp
-            /  |        transpos1      | trans2 trans3    \         /   |
+            /  |        transpose1      | trans2 trans3    \         /   |
            /   |         /    |        |    |      |        \      /    |
     softmax1   |        /     |        |    |      |          \   /     |
          \     |      /    softmax2    |    |      |          add1      |
diff --git a/test/cpp/pir/sub_graph/sub_graph_checker_test.cc b/test/cpp/pir/sub_graph/sub_graph_checker_test.cc
@@ -145,7 +145,7 @@ std::shared_ptr<::pir::Program> BuildDropOutPhiProgram() {
   return program;
 }
 
-TEST(sub_grah_checker, test_softmax) {
+TEST(sub_graph_checker, test_softmax) {
   auto basic_program = BuildBasicProgram();
   auto prim_program = BuildPrimProgram();
 
@@ -156,7 +156,7 @@ TEST(sub_grah_checker, test_softmax) {
   ASSERT_EQ(speed_data.size(), 2u);
 }
 
-TEST(sub_grah_checker, test_dropout) {
+TEST(sub_graph_checker, test_dropout) {
   auto basic_program = BuildDropOutPhiProgram();
   auto prim_program = BuildDropOutPrimProgram();
 
diff --git a/test/cpp/pir/tools/test_interface.h b/test/cpp/pir/tools/test_interface.h
@@ -45,9 +45,9 @@ class InferShapeInterface : public pir::OpInterfaceBase<InferShapeInterface> {
   template <class ConcreteOp>
   struct Model : public Concept {
     static void InferShape(pir::Operation *op) {
-      ConcreteOp concret_op = ConcreteOp(op);
-      if (concret_op == nullptr) throw("concret_op is nullptr");
-      concret_op.InferShape();
+      ConcreteOp concrete_op = ConcreteOp(op);
+      if (concrete_op == nullptr) throw("concrete_op is nullptr");
+      concrete_op.InferShape();
     }
 
     Model() : Concept(InferShape) {}
diff --git a/test/cpp/prim/test_static_prim.cc b/test/cpp/prim/test_static_prim.cc
@@ -222,7 +222,7 @@ TEST(StaticPrim, TanhBackwardComposite) {
             static_cast<std::size_t>(1));
 }
 
-TEST(StaticCompositeGradMaker, TestMutiInputMethod) {
+TEST(StaticCompositeGradMaker, TestMultiInputMethod) {
   // Initialized environment
   FLAGS_tensor_operants_mode = "static";
   paddle::OperantsManager::Instance().static_operants.reset(
@@ -266,22 +266,22 @@ TEST(StaticCompositeGradMaker, TestMutiInputMethod) {
                                      target_block,
                                      grad_sub_block);
   test();
-  std::vector<paddle::Tensor> muti_fw_input = test.GetMultiForwardInput("X");
-  paddle::optional<std::vector<paddle::Tensor>> opt_muti_fw_input =
+  std::vector<paddle::Tensor> multi_fw_input = test.GetMultiForwardInput("X");
+  paddle::optional<std::vector<paddle::Tensor>> opt_multi_fw_input =
       test.GetOptionalMultiForwardInput("X");
-  std::vector<paddle::Tensor> opt_inner = opt_muti_fw_input.is_initialized()
-                                              ? opt_muti_fw_input.get()
+  std::vector<paddle::Tensor> opt_inner = opt_multi_fw_input.is_initialized()
+                                              ? opt_multi_fw_input.get()
                                               : std::vector<paddle::Tensor>{};
   paddle::Tensor fw_out = test.GetSingleForwardOutput("Out");
   paddle::Tensor* fw_out_ptr = test.GetOutputPtr(&fw_out);
   std::string fw_out_name = test.GetOutputName(fw_out);
 
-  ASSERT_EQ(muti_fw_input.size(), static_cast<std::size_t>(2));
+  ASSERT_EQ(multi_fw_input.size(), static_cast<std::size_t>(2));
   ASSERT_EQ(
-      static_cast<prim::DescTensor*>(muti_fw_input[0].impl().get())->Name(),
+      static_cast<prim::DescTensor*>(multi_fw_input[0].impl().get())->Name(),
       "x0");
   ASSERT_EQ(
-      static_cast<prim::DescTensor*>(muti_fw_input[1].impl().get())->Name(),
+      static_cast<prim::DescTensor*>(multi_fw_input[1].impl().get())->Name(),
       "x1");
   ASSERT_EQ(opt_inner.size(), static_cast<std::size_t>(2));
   ASSERT_EQ(static_cast<prim::DescTensor*>(opt_inner[0].impl().get())->Name(),
@@ -292,7 +292,7 @@ TEST(StaticCompositeGradMaker, TestMutiInputMethod) {
   ASSERT_EQ(fw_out_name, "out");
 }
 
-TEST(StaticCompositeGradMaker, TestMutiOutputMethod) {
+TEST(StaticCompositeGradMaker, TestMultiOutputMethod) {
   // Initialized environment
   FLAGS_tensor_operants_mode = "static";
   paddle::OperantsManager::Instance().static_operants.reset(
diff --git a/test/custom_op/custom_concat_op.cc b/test/custom_op/custom_concat_op.cc
@@ -97,7 +97,7 @@ std::vector<paddle::Tensor> ConcatBackwardDynamicAxis(
   CHECK_INPUT(axis_t);
   CHECK_INPUT(grad_out);
 
-  // compate axis
+  // compute axis
   int64_t rank = static_cast<int64_t>(inputs[0].shape().size());
   int64_t axis = axis_t.data<int64_t>()[0];
   axis = ComputeAxis(axis, rank);
@@ -182,7 +182,7 @@ std::vector<paddle::Tensor> ConcatBackwardStaticAxis(
   }
   CHECK_INPUT(grad_out);
 
-  // compate axis
+  // compute axis
   int64_t rank = static_cast<int64_t>(inputs[0].shape().size());
   auto final_axis = ComputeAxis(axis, rank);
 
diff --git a/test/custom_op/custom_linear_op.cc b/test/custom_op/custom_linear_op.cc
@@ -44,10 +44,10 @@ std::vector<std::vector<int64_t>> LinearInferShape(
   auto ndims_y = weight_shape.size();
   PD_CHECK(ndims_x > 0,
            "The Input(x) dims size must be greater than 0,"
-           " but reviced dims size is 0. ");
+           " but received dims size is 0. ");
   PD_CHECK(ndims_y > 0,
            "The Input(y) dims size must be greater than 0,"
-           " but reviced dims size is 0. ");
+           " but received dims size is 0. ");
 
   bool x_broadcasted = false, y_broadcasted = false;
   if (ndims_x == 1) {
diff --git a/test/custom_op/dispatch_test_op.cc b/test/custom_op/dispatch_test_op.cc
@@ -26,7 +26,7 @@ void assign_cpu_kernel(const data_t* x_data,
   }
 }
 
-std::vector<paddle::Tensor> DispatchTestInterger(const paddle::Tensor& x) {
+std::vector<paddle::Tensor> DispatchTestInteger(const paddle::Tensor& x) {
   auto out = paddle::empty_like(x);
 
   PD_DISPATCH_INTEGRAL_TYPES(
@@ -41,7 +41,7 @@ std::vector<paddle::Tensor> DispatchTestInterger(const paddle::Tensor& x) {
 PD_BUILD_OP(dispatch_test_integer)
     .Inputs({"X"})
     .Outputs({"Out"})
-    .SetKernelFn(PD_KERNEL(DispatchTestInterger));
+    .SetKernelFn(PD_KERNEL(DispatchTestInteger));
 
 std::vector<paddle::Tensor> DispatchTestFloatAndInteger(
     const paddle::Tensor& x) {

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ void test_parameterless_type_helper() {`
`51`	`51`	`(test_parameterless_type<IR_TYPE>(), 0)...};`
`52`	`52`	`}`
`53`	`53`
`54`		`-TEST(TypeConverterTest, paramterless_type) {`
	`54`	`+TEST(TypeConverterTest, parameterless_type) {`
`55`	`55`	`test_parameterless_type_helper<pir::UInt8Type,`
`56`	`56`	`pir::Int8Type,`
`57`	`57`	`pir::BFloat16Type,`
Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ std::shared_ptr<::pir::Program> BuildDropOutPhiProgram() {`
`145`	`145`	`return program;`
`146`	`146`	`}`
`147`	`147`
`148`		`-TEST(sub_grah_checker, test_softmax) {`
	`148`	`+TEST(sub_graph_checker, test_softmax) {`
`149`	`149`	`auto basic_program = BuildBasicProgram();`
`150`	`150`	`auto prim_program = BuildPrimProgram();`
`151`	`151`
`@@ -156,7 +156,7 @@ TEST(sub_grah_checker, test_softmax) {`
`156`	`156`	`ASSERT_EQ(speed_data.size(), 2u);`
`157`	`157`	`}`
`158`	`158`
`159`		`-TEST(sub_grah_checker, test_dropout) {`
	`159`	`+TEST(sub_graph_checker, test_dropout) {`
`160`	`160`	`auto basic_program = BuildDropOutPhiProgram();`
`161`	`161`	`auto prim_program = BuildDropOutPrimProgram();`
`162`	`162`