Fix protmotion promotion (#60932)

co63oc · web-flow · commit 1e60ba9d640a · 2024-01-19T10:22:46.000+08:00
diff --git a/paddle/fluid/distributed/ps/service/ps_client.h b/paddle/fluid/distributed/ps/service/ps_client.h
@@ -233,8 +233,8 @@ class PSClient {
   // client2client消息处理，std::function<int32_t (int, int, const std::string&)
   // -> ret (msg_type, from_client_id, msg)
   typedef std::function<int32_t(int, int, const std::string &)> MsgHandlerFunc;
-  virtual int RegisteClient2ClientMsgHandler(int msg_type,
-                                             MsgHandlerFunc handler) {
+  virtual int RegisterClient2ClientMsgHandler(int msg_type,
+                                              MsgHandlerFunc handler) {
     _msg_handler_map[msg_type] = handler;
     return 0;
   }
diff --git a/paddle/fluid/distributed/ps/wrapper/fleet.cc b/paddle/fluid/distributed/ps/wrapper/fleet.cc
@@ -483,7 +483,7 @@ void FleetWrapper::PushDenseVarsAsync(
 
     ::paddle::distributed::Region reg(g, tensor->numel());
     regions.emplace_back(std::move(reg));
-    VLOG(3) << "FleetWrapper::PushDenseVarsAsync Var " << t << " talbe_id "
+    VLOG(3) << "FleetWrapper::PushDenseVarsAsync Var " << t << " table_id "
             << table_id << " Temp_data[0] " << g[0] << " Temp_data[-1] "
             << g[tensor->numel() - 1];
   }
@@ -813,7 +813,7 @@ void FleetWrapper::ShrinkDenseTable(int table_id,
   push_status.wait();
   auto status = push_status.get();
   if (status != 0) {
-    // PADDLE_THORW(platform::errors::Fatal(
+    // PADDLE_THROW(platform::errors::Fatal(
     //    "push shrink dense param failed, status is [%d].", status));
     sleep(sleep_seconds_before_fail_exit_);
     exit(-1);
@@ -839,7 +839,7 @@ int FleetWrapper::RegisterClientToClientMsgHandler(int msg_type,
     VLOG(0) << "FleetWrapper::Client is null";
     return -1;
   } else {
-    return worker_ptr_->RegisteClient2ClientMsgHandler(msg_type, handler);
+    return worker_ptr_->RegisterClient2ClientMsgHandler(msg_type, handler);
   }
 }
 
diff --git a/paddle/fluid/distributed/rpc/rpc_agent.cc b/paddle/fluid/distributed/rpc/rpc_agent.cc
@@ -138,7 +138,7 @@ void RpcAgent::SetAgentInstance(std::shared_ptr<RpcAgent> agent) {
       rpc_agent_instance_,
       nullptr,
       platform::errors::Fatal(
-          "RpcAgent has been set, please don't set rpc agent repeatly."));
+          "RpcAgent has been set, please don't set rpc agent repeatedly."));
   rpc_agent_instance_ = agent;
 }
 }  // namespace distributed
diff --git a/paddle/fluid/distributed/test/brpc_service_dense_sgd_test.cc b/paddle/fluid/distributed/test/brpc_service_dense_sgd_test.cc
@@ -192,7 +192,7 @@ void RunBrpcPushDense() {
   auto ph_host = paddle::distributed::PSHost(ip_, port_, 0);
   host_sign_list_.push_back(ph_host.SerializeToString());
 
-  // Srart Server
+  // Start Server
   std::thread server_thread(RunServer);
   sleep(1);
 
diff --git a/paddle/fluid/distributed/test/brpc_service_sparse_sgd_test.cc b/paddle/fluid/distributed/test/brpc_service_sparse_sgd_test.cc
@@ -203,7 +203,7 @@ void RunBrpcPushSparse() {
   auto ph_host = paddle::distributed::PSHost(ip_, port_, 0);
   host_sign_list_.push_back(ph_host.SerializeToString());
 
-  // Srart Server
+  // Start Server
   std::thread server_thread(RunServer);
   sleep(1);
 
diff --git a/paddle/fluid/eager/api/manual/eager_manual/forwards/multiply_fwd_func.cc b/paddle/fluid/eager/api/manual/eager_manual/forwards/multiply_fwd_func.cc
@@ -60,9 +60,9 @@ paddle::Tensor multiply_ad_func(const paddle::Tensor& x,
 
   // Type promotion Logic
   if (phi::NeedTypePromotion(x.dtype(), y.dtype())) {
-    VLOG(5) << "got different data type, run type protmotion automatically.";
+    VLOG(5) << "got different data type, run type promotion automatically.";
     LOG_FIRST_N(WARNING, 1)
-        << "got different data type, run type protmotion "
+        << "got different data type, run type promotion "
            "automatically, this may cause data type been changed.";
     auto op_name = phi::TransToFluidOpName("multiply");
     auto promotion_type = phi::GetPromoteDtype(op_name, x.dtype(), y.dtype());
@@ -407,9 +407,9 @@ paddle::Tensor multiply_ad_func(const paddle::Tensor& x,
 
   // Type promotion Logic
   if (phi::NeedTypePromotion(x.dtype(), y.dtype())) {
-    VLOG(5) << "got different data type, run type protmotion automatically.";
+    VLOG(5) << "got different data type, run type promotion automatically.";
     LOG_FIRST_N(WARNING, 1)
-        << "got different data type, run type protmotion "
+        << "got different data type, run type promotion "
            "automatically, this may cause data type been changed.";
     auto op_name = phi::TransToFluidOpName("multiply");
     auto promotion_type = phi::GetPromoteDtype(op_name, x.dtype(), y.dtype());
diff --git a/paddle/fluid/eager/api/manual/eager_manual/forwards/reshard_fwd_func.cc b/paddle/fluid/eager/api/manual/eager_manual/forwards/reshard_fwd_func.cc
@@ -82,7 +82,7 @@ paddle::Tensor reshard_ad_function(
 #else
   PADDLE_THROW(phi::errors::Unavailable(
       "Reshard is not supported in this version of Paddle. Try to recompile it "
-      "with WITH_DISTRIBTUE=ON and reinstall this package."));
+      "with WITH_DISTRIBUTE=ON and reinstall this package."));
   return paddle::Tensor();
 #endif
 }
diff --git a/paddle/fluid/eager/api/manual/eager_manual/nodes/reshard_node.cc b/paddle/fluid/eager/api/manual/eager_manual/nodes/reshard_node.cc
@@ -99,7 +99,7 @@ ReshardGradNode::operator()(
 #else
   PADDLE_THROW(phi::errors::Unavailable(
       "ReshardGrad is not supported in this version of Paddle. Try to "
-      "recompile it with WITH_DISTRIBTUE=ON and reinstall this package."));
+      "recompile it with WITH_DISTRIBUTE=ON and reinstall this package."));
   return paddle::small_vector<std::vector<paddle::Tensor>,
                               egr::kSlotSmallVectorSize>(1);
 #endif
diff --git a/paddle/fluid/eager/auto_code_generator/eager_generator.cc b/paddle/fluid/eager/auto_code_generator/eager_generator.cc
@@ -586,7 +586,7 @@ static bool CheckOpProto(proto::OpProto* op_proto) {
   }
   const std::string& op_type = op_proto->type();
 
-  // Skip ooerator which is not inherit form OperatorWithKernel, like while,
+  // Skip operator which is not inherit form OperatorWithKernel, like while,
   // since only OperatorWithKernel can run in dygraph mode.
   auto& all_kernels = paddle::framework::OperatorWithKernel::AllOpKernels();
   if (!all_kernels.count(op_type) &&
@@ -2702,7 +2702,7 @@ static std::string GenerateGradNodeCCContents(
   // This is a Copy
   auto op_base_infos = bwd_info.GetOpBaseInfos();
 
-  /* Special Case: ops such as sum_grad_op is implemented abnormaly,
+  /* Special Case: ops such as sum_grad_op is implemented abnormally,
                    where it unpacked duplicable GradX and created one OpBase
                    corresponds to each member of GradX[i]
      */
diff --git a/paddle/fluid/eager/custom_operator/custom_operator_utils.cc b/paddle/fluid/eager/custom_operator/custom_operator_utils.cc
@@ -213,7 +213,7 @@ static std::vector<std::vector<phi::DDim>> RunInferShapeFunc(
               "Custom operator only supports `paddle::Vec(...)` inputs and "
               "cannot support `paddle::Vec(...)` output without setting "
               "InplaceMap. If you have to use `paddle::Vec(...)` output, "
-              "please indicate it by setting InplaceMap manully."));
+              "please indicate it by setting InplaceMap manually."));
       std::vector<phi::DDim> shapes;
       auto duplicable_input_pair = ctx.InputRangeAt(inplace_reverse_map[i]);
       for (size_t j = duplicable_input_pair.first;
@@ -570,7 +570,7 @@ std::vector<std::vector<phi::DDim>> RunInferShapeFn(
       out_dims.size(),
       ctx.OutputRange().size(),
       phi::errors::InvalidArgument(
-          "Custome op infer_shape return size should be %d, but got %d.",
+          "Custom op infer_shape return size should be %d, but got %d.",
           ctx.OutputRange().size(),
           out_dims.size()));
 
@@ -603,7 +603,7 @@ std::vector<std::vector<phi::DataType>> RunInferDtypeFn(
       out_dtypes.size(),
       ctx.OutputRange().size(),
       phi::errors::InvalidArgument(
-          "Custome op infer_dtype return size should be %d, but got %d.",
+          "Custom op infer_dtype return size should be %d, but got %d.",
           ctx.OutputRange().size(),
           out_dtypes.size()));
   return out_dtypes;
@@ -677,15 +677,15 @@ std::
       PADDLE_ENFORCE_EQ(
           out_dim.size(),
           pair.second - pair.first,
-          phi::errors::InvalidArgument("custome op infer_shape result[%d]'s "
+          phi::errors::InvalidArgument("custom op infer_shape result[%d]'s "
                                        "size should be %d, but got %d.",
                                        i,
                                        pair.second - pair.first,
                                        out_dim.size()));
       PADDLE_ENFORCE_EQ(
           out_dtype.size(),
           pair.second - pair.first,
-          phi::errors::InvalidArgument("custome op infer_shape result[%d]'s "
+          phi::errors::InvalidArgument("custom op infer_shape result[%d]'s "
                                        "size should be %d, but got %d.",
                                        i,
                                        pair.second - pair.first,

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ void RpcAgent::SetAgentInstance(std::shared_ptr<RpcAgent> agent) {`
`138`	`138`	`rpc_agent_instance_,`
`139`	`139`	`nullptr,`
`140`	`140`	`platform::errors::Fatal(`
`141`		`- "RpcAgent has been set, please don't set rpc agent repeatly."));`
	`141`	`+ "RpcAgent has been set, please don't set rpc agent repeatedly."));`
`142`	`142`	`rpc_agent_instance_ = agent;`
`143`	`143`	`}`
`144`	`144`	`} // namespace distributed`
Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ paddle::Tensor reshard_ad_function(`
`82`	`82`	`#else`
`83`	`83`	`PADDLE_THROW(phi::errors::Unavailable(`
`84`	`84`	`"Reshard is not supported in this version of Paddle. Try to recompile it "`
`85`		`- "with WITH_DISTRIBTUE=ON and reinstall this package."));`
	`85`	`+ "with WITH_DISTRIBUTE=ON and reinstall this package."));`
`86`	`86`	`return paddle::Tensor();`
`87`	`87`	`#endif`
`88`	`88`	`}`