Fix some typos (excute_cmd_result, specied, etc.) (#61760)

co63oc · web-flow · commit 3611c08e86e7 · 2024-02-20T14:52:36.000+08:00
diff --git a/paddle/fluid/framework/fleet/heter_ps/heter_resource.cc b/paddle/fluid/framework/fleet/heter_ps/heter_resource.cc
@@ -124,7 +124,7 @@ void HeterPsResource::enable_p2p() {
   }
 #endif
 }
-static std::string excute_cmd_result(const std::string &cmd) {
+static std::string execute_cmd_result(const std::string &cmd) {
   FILE *fp = popen(cmd.c_str(), "r");
   if (fp == NULL) {
     fprintf(stderr, "cmd %s open failed\n", cmd.c_str());
@@ -171,7 +171,8 @@ bool GpuRDMAChecker::check_device_status(const int &device_count,
     return false;
   }
   // a100
-  std::string str = excute_cmd_result("source ~/.bashrc && nvidia-smi topo -m");
+  std::string str =
+      execute_cmd_result("source ~/.bashrc && nvidia-smi topo -m");
   if (str.empty()) {  // a100 auto gpu card rdma status
     return false;
   }
diff --git a/paddle/fluid/framework/fleet/metrics.cc b/paddle/fluid/framework/fleet/metrics.cc
@@ -327,7 +327,7 @@ void BasicAucCalculator::computeWuAuc() {
     if (wuauc_records_[i].uid_ != prev_uid) {
       std::vector<WuaucRecord> single_user_recs(
           wuauc_records_.begin() + prev_pos, wuauc_records_.begin() + i);
-      roc_data = computeSingelUserAuc(single_user_recs);
+      roc_data = computeSingleUserAuc(single_user_recs);
       if (roc_data.auc_ != -1) {
         double ins_num = (roc_data.tp_ + roc_data.fp_);
         _user_cnt += 1;
@@ -343,7 +343,7 @@ void BasicAucCalculator::computeWuAuc() {
 
   std::vector<WuaucRecord> single_user_recs(wuauc_records_.begin() + prev_pos,
                                             wuauc_records_.end());
-  roc_data = computeSingelUserAuc(single_user_recs);
+  roc_data = computeSingleUserAuc(single_user_recs);
   if (roc_data.auc_ != -1) {
     double ins_num = (roc_data.tp_ + roc_data.fp_);
     _user_cnt += 1;
@@ -353,7 +353,7 @@ void BasicAucCalculator::computeWuAuc() {
   }
 }
 
-BasicAucCalculator::WuaucRocData BasicAucCalculator::computeSingelUserAuc(
+BasicAucCalculator::WuaucRocData BasicAucCalculator::computeSingleUserAuc(
     const std::vector<WuaucRecord>& records) {
   double tp = 0.0;
   double fp = 0.0;
diff --git a/paddle/fluid/framework/fleet/metrics.h b/paddle/fluid/framework/fleet/metrics.h
@@ -86,7 +86,7 @@ class BasicAucCalculator {
 
   void compute();
   void computeWuAuc();
-  WuaucRocData computeSingelUserAuc(const std::vector<WuaucRecord>& records);
+  WuaucRocData computeSingleUserAuc(const std::vector<WuaucRecord>& records);
   int table_size() const { return _table_size; }
   double bucket_error() const { return _bucket_error; }
   double auc() const { return _auc; }
diff --git a/paddle/fluid/framework/fleet/ps_gpu_wrapper.cc b/paddle/fluid/framework/fleet/ps_gpu_wrapper.cc
@@ -2323,7 +2323,7 @@ void PSGPUWrapper::PullSparse(const paddle::platform::Place& place,
                               const std::vector<int64_t>& slot_lengths,
                               const std::vector<int>& slot_dim,
                               const int hidden_size) {
-  VLOG(3) << "Begine Gpu Ps PullSparse";
+  VLOG(3) << "Begin Gpu Ps PullSparse";
   platform::Timer all_timer;
   platform::Timer pull_gpups_timer;
   all_timer.Start();
@@ -2565,7 +2565,7 @@ void PSGPUWrapper::PullSparse(const paddle::platform::Place& place,
 #endif
   } else if (platform::is_xpu_place(place)) {
 #ifdef PADDLE_WITH_XPU_KP
-    VLOG(3) << "Begine Xpu Ps PullSparse";
+    VLOG(3) << "Begin Xpu Ps PullSparse";
     size_t total_length =
         std::accumulate(slot_lengths.begin(), slot_lengths.end(), 0UL);
     FeatureValue* total_values_gpu = nullptr;
diff --git a/paddle/fluid/framework/io/crypto/cipher.cc b/paddle/fluid/framework/io/crypto/cipher.cc
@@ -50,7 +50,7 @@ std::shared_ptr<Cipher> CipherFactory::CreateCipher(
     return ret;
   } else {
     PADDLE_THROW(paddle::platform::errors::InvalidArgument(
-        "Invalid cipher name is specied. "
+        "Invalid cipher name is specified. "
         "Please check you have specified valid cipher"
         " name in CryptoProperties."));
   }
diff --git a/paddle/fluid/framework/ir/fusion_group/code_generator.cc b/paddle/fluid/framework/ir/fusion_group/code_generator.cc
@@ -210,7 +210,7 @@ std::string CodeGenerator::Generate(
 std::set<int> CodeGenerator::DistilInputIds(
     const std::vector<OperationExpression>& expressions) {
   std::set<int> input_ids;
-  // Use std::set to remove the reptead id and get a ordered list.
+  // Use std::set to remove the repeated id and get a ordered list.
   for (const auto& expression : expressions) {
     for (auto id : expression.GetInputIds()) {
       if (id >= 0) {
@@ -224,7 +224,7 @@ std::set<int> CodeGenerator::DistilInputIds(
 std::set<int> CodeGenerator::DistilOutputIds(
     const std::vector<OperationExpression>& expressions) {
   std::set<int> output_ids;
-  // Use std::set to remove the reptead id and get a ordered list.
+  // Use std::set to remove the repeated id and get a ordered list.
   for (const auto& expression : expressions) {
     for (auto id : expression.GetOutputIds()) {
       output_ids.insert(id);
@@ -236,7 +236,7 @@ std::set<int> CodeGenerator::DistilOutputIds(
 std::set<int> CodeGenerator::DistilIntermediateIds(
     const std::vector<OperationExpression>& expressions) {
   std::set<int> intermediate_output_ids;
-  // Use std::set to remove the reptead id and get a ordered list.
+  // Use std::set to remove the repeated id and get a ordered list.
   for (const auto& expression : expressions) {
     for (auto id : expression.GetIntermediateOutputIds()) {
       intermediate_output_ids.insert(id);
diff --git a/paddle/fluid/framework/ir/fusion_group/operation.cc b/paddle/fluid/framework/ir/fusion_group/operation.cc
@@ -96,8 +96,8 @@ void OperationMap::InsertUnaryElementwiseOperations() {
                             std::string expr,
                             std::vector<std::string> grad_exprs) {
     int type = 0;
-    int num_oprands = 1;
-    Insert(type, num_oprands, op_type, expr, grad_exprs, {"X"}, {"Out"});
+    int num_operands = 1;
+    Insert(type, num_operands, op_type, expr, grad_exprs, {"X"}, {"Out"});
   };
 
   // relu:
diff --git a/paddle/fluid/framework/ir/ipu/forward_graph_extract_pass.cc b/paddle/fluid/framework/ir/ipu/forward_graph_extract_pass.cc
@@ -115,7 +115,7 @@ void ForwardGraphExtractPass::ApplyImpl(ir::Graph* graph) const {
 
   VLOG(10) << "Remove Node: ";
   for (auto* node : rm_nodes) {
-    // rm node releations
+    // rm node relations
     for (auto* node_in : node->inputs) {
       for (size_t i = 0; i < node_in->outputs.size(); ++i) {
         if (node_in->outputs[i] == node) {
diff --git a/paddle/fluid/framework/ir/memory_optimize_pass/conditional_block_op_eager_deletion_pass.cc b/paddle/fluid/framework/ir/memory_optimize_pass/conditional_block_op_eager_deletion_pass.cc
@@ -45,7 +45,7 @@ class ConditionalOpEagerDeletionPass : public Pass {
     }
 
     // NOTE(Aurelius84): In case of @to_static, after we finish executing
-    // forward graph, some necessaray variable in step_scope of controlflow_op
+    // forward graph, some necessary variable in step_scope of controlflow_op
     // should be kept for backward graph.
     if (graph->IsConstructedByPartialProgram()) {
       PADDLE_ENFORCE_LE(target_ops.size(),
diff --git a/paddle/fluid/framework/ir/memory_optimize_pass/inplace_addto_op_pass.cc b/paddle/fluid/framework/ir/memory_optimize_pass/inplace_addto_op_pass.cc
@@ -137,7 +137,7 @@ void InplaceAddToOpPass::Run(Graph *graph) const {
     auto *op = op_vars_pair.first;
 
     // The original gradient accumulation is g = sum(g_0, g_1,..., g_n), and it
-    // could be changed as follws if inplace addto is enabled:
+    // could be changed as follows if inplace addto is enabled:
     // g_sum_0 = g_0
     // g_sum_1 = grad_add(g_sum_0, g_1)
     // g_sum_2 = grad_add(g_sum_1, g_2)
diff --git a/paddle/fluid/framework/ir/memory_optimize_pass/pylayer_op_eager_deletion_pass.cc b/paddle/fluid/framework/ir/memory_optimize_pass/pylayer_op_eager_deletion_pass.cc
@@ -45,7 +45,7 @@ class PyLayerOpEagerDeletionPass : public Pass {
     }
 
     // NOTE(Aurelius84): In case of @to_static, after we finish executing
-    // forward graph, some necessaray variable in step_scope of pylayer_op
+    // forward graph, some necessary variable in step_scope of pylayer_op
     // should be kept for backward graph.
     if (graph->IsConstructedByPartialProgram()) {
       PADDLE_ENFORCE_LE(target_ops.size(),
diff --git a/paddle/fluid/framework/ir/mkldnn/cpu_bfloat16_pass_tester.cc b/paddle/fluid/framework/ir/mkldnn/cpu_bfloat16_pass_tester.cc
@@ -127,7 +127,7 @@ ProgramDesc BuildProgramDescDoubleInput(bool use_mkldnn) {
   SetOp(&prog, "transpose2", "Transpose", {"d"}, {"e"}, use_mkldnn, "float32");
   SetOp(&prog,
         "elementwise_add",
-        "ElemetwiseAdd",
+        "ElementwiseAdd",
         {"c", "e"},
         {"f"},
         use_mkldnn,
diff --git a/paddle/fluid/framework/ir/xpu/xpu_quantize_squash_pass.h b/paddle/fluid/framework/ir/xpu/xpu_quantize_squash_pass.h
@@ -56,7 +56,7 @@ class XPUQuantizeSquashPass : public FusePassBase {
   void OpDequantSquash(Graph* graph) const;
 
   /*
-   * Squash quantize if several quatize ops have the same scale
+   * Squash quantize if several quantize ops have the same scale
    */
   void MultipleQuantizeSquash(Graph* graph) const;
 
diff --git a/paddle/fluid/framework/section_worker.cc b/paddle/fluid/framework/section_worker.cc
@@ -86,7 +86,7 @@ void SectionWorker::Initialize(const TrainerDesc &desc) {
 }
 
 void SectionWorker::PrepareUnusedVar() {
-  VLOG(5) << "begin prepare the unsed vars";
+  VLOG(5) << "begin prepare the unused vars";
   unused_vars_ = GetUnusedVars(program_->Block(0), ops_, skip_vars_);
 }
 
@@ -156,7 +156,7 @@ void SectionWorker::RunFThenB(std::unique_ptr<GarbageCollector> &gc) {
 }
 
 void SectionWorker::Run1F1B(std::unique_ptr<GarbageCollector> &gc) {
-  // 1F1B scheduler, which runs forward phase and backward phase altertively
+  // 1F1B scheduler, which runs forward phase and backward phase alternatively
   // after startup phase. For a stage, the number of microbatches for
   // startup is num_pipeline_stages_ - pipeline_stage_ - 1, where
   // num_pipeline_stages_ is the total number of pipeline stages and
diff --git a/paddle/fluid/framework/selected_rows_utils.h b/paddle/fluid/framework/selected_rows_utils.h
@@ -28,8 +28,8 @@ limitations under the License. */
 namespace paddle {
 namespace framework {
 /*
- * Serialize/Desiralize SelectedRows to std::ostream
- * You can pass ofstream or ostringstream to serilize to file
+ * Serialize/Deserialize SelectedRows to std::ostream
+ * You can pass ofstream or ostringstream to serialize to file
  * or to a in memory string. GPU tensor will be copied to CPU.
  */
 void SerializeToStream(std::ostream& os,
diff --git a/paddle/fluid/framework/var_type_inference.h b/paddle/fluid/framework/var_type_inference.h
@@ -220,7 +220,7 @@ class InferVarTypeContext {
     this->SetVarLoDLevel(var_name, lod_level);
   }
 
-  // add a speical API for save_op
+  // add a special API for save_op
   // avoid use this API for common logic
   virtual void InsertVar(const std::string& var_name,
                          proto::VarType::Type var_type) {
diff --git a/paddle/fluid/framework/version.h b/paddle/fluid/framework/version.h
@@ -39,7 +39,7 @@ constexpr int64_t kCurProgramVersion = 0;
 // paddle in 2.4.2 and before does not support Scalar in op attributes
 // and no op in program of 2.4.2 or earlier versions has attributes of type
 // `Scalar` This version number is used for converting program to a legacy
-// format to indentifiy their version id See Also
+// format to identify their version id. See Also
 // paddle/fluid/framework/op_version_proto.cc
 constexpr int64_t kLegacyProgramVersion = 2004002L;
 
diff --git a/paddle/fluid/imperative/prepared_operator.h b/paddle/fluid/imperative/prepared_operator.h
@@ -489,7 +489,7 @@ void BuildDygraphPhiKernelContext(const phi::KernelSignature& kernel_signature,
         PADDLE_ENFORCE_NOT_NULL(
             attr_ptr,
             platform::errors::NotFound("(%s) is not found in AttributeMap when "
-                                       "buildind dygraph KernelContext.",
+                                       "building dygraph KernelContext.",
                                        attr_names[i]));
         auto& attr = *attr_ptr;
         switch (AttrTypeID(attr)) {
diff --git a/paddle/fluid/imperative/tracer.cc b/paddle/fluid/imperative/tracer.cc
@@ -96,7 +96,7 @@ const std::shared_ptr<AMPState>& GetCurrentAMPState() {
 void PassStopGradient(const NameVarBaseMap& outs, bool generate_grad) {
   for (const auto& pair : outs) {
     for (const auto& var : pair.second) {
-      // NOTE(zhiqiu): this happends when None output are passed from python
+      // NOTE(zhiqiu): this happens when None output are passed from python
       // side. For example, fake_quantize_dequantize_moving_average_abs_max may
       // pass None OutAccum in eval mode.
       // It can be refined by generate several different pybind interface for
@@ -504,7 +504,7 @@ void Tracer::TraceOp(const std::string& type,
                      const NameTensorMap& ins,
                      const NameTensorMap& outs,
                      paddle::framework::AttributeMap attrs) {
-  VLOG(6) << "Running On Eager TraceOp(4 agrs): ";
+  VLOG(6) << "Running On Eager TraceOp(4 args): ";
   TraceOpImpl<egr::EagerVariable>(
       type, ins, outs, attrs, expected_place_, false, {}, nullptr, true);
 }
diff --git a/paddle/fluid/inference/io.cc b/paddle/fluid/inference/io.cc
@@ -86,7 +86,7 @@ void LoadPersistables(framework::Executor* executor,
 
   framework::ProgramDesc* load_program = new framework::ProgramDesc();
   framework::BlockDesc* load_block = load_program->MutableBlock(0);
-  std::vector<std::string> paramlist;
+  std::vector<std::string> param_list;
 
   for (auto* var : global_block.AllVars()) {
     if (IsPersistable(var)) {
@@ -107,7 +107,7 @@ void LoadPersistables(framework::Executor* executor,
       new_var->SetPersistable(true);
 
       if (!param_filename.empty()) {
-        paramlist.push_back(new_var->Name());
+        param_list.push_back(new_var->Name());
       } else {
         // append_op
         framework::OpDesc* op = load_block->AppendOp();
@@ -120,12 +120,12 @@ void LoadPersistables(framework::Executor* executor,
   }
 
   if (!param_filename.empty()) {
-    // sort paramlist to have consistent ordering
-    std::sort(paramlist.begin(), paramlist.end());
+    // sort param_list to have consistent ordering
+    std::sort(param_list.begin(), param_list.end());
     // append just the load_combine op
     framework::OpDesc* op = load_block->AppendOp();
     op->SetType("load_combine");
-    op->SetOutput("Out", paramlist);
+    op->SetOutput("Out", param_list);
     op->SetAttr("file_path", {param_filename});
     op->SetAttr("model_from_memory", {model_from_memory});
     op->CheckAttrs();

Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,7 @@ void HeterPsResource::enable_p2p() {`
`124`	`124`	`}`
`125`	`125`	`#endif`
`126`	`126`	`}`
`127`		`-static std::string excute_cmd_result(const std::string &cmd) {`
	`127`	`+static std::string execute_cmd_result(const std::string &cmd) {`
`128`	`128`	`FILE *fp = popen(cmd.c_str(), "r");`
`129`	`129`	`if (fp == NULL) {`
`130`	`130`	`fprintf(stderr, "cmd %s open failed\n", cmd.c_str());`
`@@ -171,7 +171,8 @@ bool GpuRDMAChecker::check_device_status(const int &device_count,`
`171`	`171`	`return false;`
`172`	`172`	`}`
`173`	`173`	`// a100`
`174`		`- std::string str = excute_cmd_result("source ~/.bashrc && nvidia-smi topo -m");`
	`174`	`+ std::string str =`
	`175`	`+ execute_cmd_result("source ~/.bashrc && nvidia-smi topo -m");`
`175`	`176`	`if (str.empty()) { // a100 auto gpu card rdma status`
`176`	`177`	`return false;`
`177`	`178`	`}`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ std::shared_ptr<Cipher> CipherFactory::CreateCipher(`
`50`	`50`	`return ret;`
`51`	`51`	`} else {`
`52`	`52`	`PADDLE_THROW(paddle::platform::errors::InvalidArgument(`
`53`		`- "Invalid cipher name is specied. "`
	`53`	`+ "Invalid cipher name is specified. "`
`54`	`54`	`"Please check you have specified valid cipher"`
`55`	`55`	`" name in CryptoProperties."));`
`56`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ class ConditionalOpEagerDeletionPass : public Pass {`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`// NOTE(Aurelius84): In case of @to_static, after we finish executing`
`48`		`- // forward graph, some necessaray variable in step_scope of controlflow_op`
	`48`	`+ // forward graph, some necessary variable in step_scope of controlflow_op`
`49`	`49`	`// should be kept for backward graph.`
`50`	`50`	`if (graph->IsConstructedByPartialProgram()) {`
`51`	`51`	`PADDLE_ENFORCE_LE(target_ops.size(),`