Fix typos usefull_op useful_op (#70700)

co63oc · web-flow · commit 5089b3bd0b98 · 2025-01-23T10:18:01.000+08:00
diff --git a/paddle/fluid/framework/new_executor/interpreter/dependency_builder.cc b/paddle/fluid/framework/new_executor/interpreter/dependency_builder.cc
@@ -1315,37 +1315,37 @@ std::vector<size_t> DependencyBuilderSimplify::get_new_executor_order() {
   std::vector<size_t> adam_vector;
   std::priority_queue<std::pair<size_t, size_t>> adam_pq;
   const std::string push_op = "push";
-  std::vector<bool> usefull_op(op_num_, false);
+  std::vector<bool> useful_op(op_num_, false);
   for (size_t i = start_index_; i < op_num_; i++) {
     int op_role = _ops_ptr->at(i)->Attr<int>("op_role");
     std::string op_name = _ops_ptr->at(i)->Type();
     if (op_role == static_cast<int>(OpRole::kOptimize) ||
         op_name.find(push_op) != std::string::npos) {
       adam_vector.push_back(i);
-      usefull_op[i] = true;
+      useful_op[i] = true;
       adam_pq.push(std::make_pair(-op_before_num[i], i));
     }
   }
   for (size_t i = start_index_; i < op_num_; i++) {
     for (auto j : adam_vector)
       if (op_happens_before_[i][j]) {
-        usefull_op[i] = true;
+        useful_op[i] = true;
         break;
       }
   }
-  std::set<size_t> not_usefull_op;
+  std::set<size_t> not_useful_op;
   for (size_t i = start_index_; i < op_num_; i++) {
-    if (usefull_op[i] == false) {
-      not_usefull_op.insert(i);
+    if (useful_op[i] == false) {
+      not_useful_op.insert(i);
       if (FLAGS_enable_dependency_builder_debug_info) {
-        VLOG(0) << "not usefull op " << _ops_ptr->at(i)->Type() << "_" << i;
+        VLOG(0) << "not useful op " << _ops_ptr->at(i)->Type() << "_" << i;
       }
     }
   }
   for (auto del_op : del_c_sync_comm_list) {
-    if (not_usefull_op.count(del_op) == 0) {
+    if (not_useful_op.count(del_op) == 0) {
       if (FLAGS_enable_dependency_builder_debug_info) {
-        VLOG(0) << " error " << del_op << " not in usefull_op";
+        VLOG(0) << " error " << del_op << " not in useful_op";
       }
     }
   }
@@ -1379,14 +1379,14 @@ std::vector<size_t> DependencyBuilderSimplify::get_new_executor_order() {
     auto current = s.top();
     s.pop();
     if (is_visit[current] == false) {
-      if (!not_usefull_op.count(current)) {
+      if (!not_useful_op.count(current)) {
         new_order.push_back(current);
       }
       is_visit[current] = true;
       for (auto it = op_downstream_map_[current].rbegin();
            it != op_downstream_map_[current].rend();
            it++) {
-        if (--dependency_count[*it] == 0 && !not_usefull_op.count(current)) {
+        if (--dependency_count[*it] == 0 && !not_useful_op.count(current)) {
           pq.push(std::make_pair(op_behind_num[*it], *it));
           // s.push(*it);
         }
@@ -1401,7 +1401,7 @@ std::vector<size_t> DependencyBuilderSimplify::get_new_executor_order() {
 
   PADDLE_ENFORCE_EQ(
       new_order.size(),
-      op_num_ - not_usefull_op.size(),
+      op_num_ - not_useful_op.size(),
       common::errors::AlreadyExists("new_order size not equal op num"));
   if (FLAGS_enable_dependency_builder_debug_info) {
     std::stringstream ss;
diff --git a/paddle/fluid/framework/new_executor/interpreter/dependency_builder.h b/paddle/fluid/framework/new_executor/interpreter/dependency_builder.h
@@ -90,7 +90,7 @@ class DependencyBuilder {
 
   // ops_behind_ is the adjacency list about op to its posterior-ops, that is to
   // say, op_behind_[i] == {a, b, c} means op[a], op[b] and op[c] depend on
-  // op[i] directly or indirectly. ops_before_ is the revered adjacency list of
+  // op[i] directly or indirectly. ops_before_ is the reversed adjacency list of
   // ops_behind_.
   std::vector<std::vector<size_t>> ops_before_;
   std::vector<std::vector<size_t>> ops_behind_;
@@ -186,7 +186,7 @@ class DependencyBuilderSimplify {
 
   // ops_behind_ is the adjacency list about op to its posterior-ops, that is to
   // say, op_behind_[i] == {a, b, c} means op[a], op[b] and op[c] depend on
-  // op[i] directly or indirectly. ops_before_ is the revered adjacency list of
+  // op[i] directly or indirectly. ops_before_ is the reversed adjacency list of
   // ops_behind_.
   std::vector<std::vector<size_t>> ops_before_;
   std::vector<std::vector<size_t>> ops_behind_;
diff --git a/paddle/fluid/inference/analysis/ir_passes/subgraph_util.cc b/paddle/fluid/inference/analysis/ir_passes/subgraph_util.cc
@@ -30,7 +30,7 @@ using framework::ir::Node;
 std::vector<std::string> ExtractParameters(
     const std::unordered_set<Node *> &nodes, bool sorted) {
   // We can judge whether a variable is a parameter by
-  // its presistable property, but sometimes the presistable
+  // its persistable property, but sometimes the persistable
   // of the feed op output is true, so we have to identify it.
   std::vector<std::string> feed_outputs;
   for (const auto &node : nodes) {
diff --git a/paddle/fluid/inference/tensorrt/convert/conv3d_op.cc b/paddle/fluid/inference/tensorrt/convert/conv3d_op.cc
@@ -33,7 +33,7 @@ void ConvertConv3d(TensorRTEngine* engine,
   auto* Y_v = scope.FindVar(filter_var_name);
   PADDLE_ENFORCE_NOT_NULL(
       Y_v,
-      common::errors::NotFound("Can not find %s presistable var in scope.",
+      common::errors::NotFound("Can not find %s persistable var in scope.",
                                filter_var_name));
   auto* Y_t = Y_v->GetMutable<phi::DenseTensor>();
   bool enable_int8 = op_desc.HasAttr("enable_int8");
diff --git a/paddle/fluid/inference/tensorrt/convert/dequantize_linear_op.cc b/paddle/fluid/inference/tensorrt/convert/dequantize_linear_op.cc
@@ -30,7 +30,7 @@ class DequantizeLinearOpConverter : public OpConverter {
     // Create constant layer for scale
     PADDLE_ENFORCE_NOT_NULL(
         scale_var,
-        common::errors::NotFound("Can not find %s presistable var in scope.",
+        common::errors::NotFound("Can not find %s persistable var in scope.",
                                  op_desc.Input("Scale")[0]));
     auto* scale_t = scale_var->GetMutable<phi::DenseTensor>();
     int n_scale = scale_t->numel();
diff --git a/paddle/fluid/inference/tensorrt/convert/emb_eltwise_layernorm.cc b/paddle/fluid/inference/tensorrt/convert/emb_eltwise_layernorm.cc
@@ -25,7 +25,7 @@ class EmbEltwiseLayerNormOpConverter : public OpConverter {
                   const framework::Scope& scope,
                   bool test_mode) override {
     VLOG(4) << "convert EmbEltwiseLayerNorm op to tensorrt layer";
-    // get the presistable var's data
+    // get the persistable var's data
     auto GetWeight = [&](const std::string& var_name,
                          phi::DDim* dim) -> TensorRTEngine::Weight {
       auto* temp_var = scope.FindVar(var_name);
diff --git a/paddle/fluid/inference/tensorrt/convert/group_norm_op.cc b/paddle/fluid/inference/tensorrt/convert/group_norm_op.cc
@@ -40,7 +40,7 @@ class GroupNormOpConverter : public OpConverter {
       with_silu = PADDLE_GET_CONST(bool, op_desc.GetAttr("with_silu"));
     }
 
-    // get the presistable var's data
+    // get the persistable var's data
     auto GetWeight = [&](const std::string& var_name,
                          phi::DDim* dims) -> TensorRTEngine::Weight {
       auto* temp_var = scope.FindVar(var_name);
diff --git a/paddle/fluid/inference/tensorrt/convert/preln_emb_eltwise_layernorm.cc b/paddle/fluid/inference/tensorrt/convert/preln_emb_eltwise_layernorm.cc
@@ -23,7 +23,7 @@ class PrelnEmbEltwiseLayerNormOpConverter : public OpConverter {
                   bool test_mode) override {
 #if IS_TRT_VERSION_GE(7000)
     VLOG(4) << "convert PrelnEmbEltwiseLayerNorm op to tensorrt layer";
-    // get the presistable var's data
+    // get the persistable var's data
     auto GetWeight = [&](const std::string& var_name,
                          phi::DDim* dim) -> TensorRTEngine::Weight {
       auto* temp_var = scope.FindVar(var_name);
diff --git a/paddle/fluid/inference/tensorrt/convert/preln_groupnorm_act_op.cc b/paddle/fluid/inference/tensorrt/convert/preln_groupnorm_act_op.cc
@@ -39,7 +39,7 @@ class PrelnGroupnormActOpConverter : public OpConverter {
     std::string scale_name = op_desc.Input("Scale").front();
     std::string bias_name = op_desc.Input("Bias").front();
 
-    // get the presistable var's data
+    // get the persistable var's data
     auto GetWeight = [&](const std::string& var_name,
                          phi::DDim* dims) -> TensorRTEngine::Weight {
       auto* temp_var = scope.FindVar(var_name);
diff --git a/paddle/fluid/inference/tensorrt/convert/prompt_tuning_emb_eltwise_layernorm.cc b/paddle/fluid/inference/tensorrt/convert/prompt_tuning_emb_eltwise_layernorm.cc
@@ -25,7 +25,7 @@ class PromptTuningEmbEltwiseLayerNormOpConverter : public OpConverter {
                   bool test_mode) override {
     VLOG(4) << "convert fused_prompt_tuning_embedding_eltwise_layernorm op to "
                "tensorrt layer";
-    // get the presistable var's data
+    // get the persistable var's data
     auto GetWeight = [&](const std::string& var_name,
                          phi::DDim* dim) -> TensorRTEngine::Weight {
       auto* temp_var = scope.FindVar(var_name);
diff --git a/paddle/fluid/inference/tensorrt/convert/quantize_linear_op.cc b/paddle/fluid/inference/tensorrt/convert/quantize_linear_op.cc
@@ -31,7 +31,7 @@ class QuantizeLinearOpConverter : public OpConverter {
     // Create constant layer for scale
     PADDLE_ENFORCE_NOT_NULL(
         scale_var,
-        common::errors::NotFound("Can not find %s presistable var in scope.",
+        common::errors::NotFound("Can not find %s persistable var in scope.",
                                  op_desc.Input("Scale")[0]));
     auto* scale_t = scale_var->GetMutable<phi::DenseTensor>();
     int n_scale = scale_t->numel();
diff --git a/paddle/fluid/inference/tensorrt/convert/skip_groupnorm_act_op.cc b/paddle/fluid/inference/tensorrt/convert/skip_groupnorm_act_op.cc
@@ -38,7 +38,7 @@ class SkipGroupnormActOpConverter : public OpConverter {
     std::string scale_name = op_desc.Input("Scale").front();
     std::string bias_name = op_desc.Input("Bias").front();
 
-    // get the presistable var's data
+    // get the persistable var's data
     auto GetWeight = [&](const std::string& var_name,
                          phi::DDim* dims) -> TensorRTEngine::Weight {
       auto* temp_var = scope.FindVar(var_name);
diff --git a/paddle/fluid/inference/tensorrt/convert/sparse_fc_op.cc b/paddle/fluid/inference/tensorrt/convert/sparse_fc_op.cc
@@ -116,7 +116,7 @@ class SparseFcOpConverter : public OpConverter {
     PADDLE_ENFORCE_NOT_NULL(
         Y_v,
         common::errors::NotFound(
-            "Can not find %s presistable var of sparse_fc in scope.", w_name));
+            "Can not find %s persistable var of sparse_fc in scope.", w_name));
     auto* Y_t = Y_v->GetMutable<phi::DenseTensor>();
     int x_num_col_dims =
         op_desc.HasAttr("x_num_col_dims")
diff --git a/paddle/fluid/pir/transforms/general/auto_layout_insert_pass.cc b/paddle/fluid/pir/transforms/general/auto_layout_insert_pass.cc
@@ -293,7 +293,7 @@ class AutoLayoutInsertPass : public pir::Pass {
 
     for (auto& operand : op->operands()) {
       if (!JudgeValue(operand.source())) continue;
-      // Canbe optimize with cache when not eliminate the transpose op.
+      // Can be optimize with cache when not eliminate the transpose op.
       auto transpose_op = builder.Build<paddle::dialect::TransposeOp>(
           operand.source(), NCHW2NHWC_);
       transpose_op->set_attribute(
diff --git a/paddle/fluid/platform/tensorrt/engine.h b/paddle/fluid/platform/tensorrt/engine.h
@@ -292,7 +292,7 @@ class TensorRTEngine {
     }
   }
 
-  // NOTE: The func bellow was modified to adapt the dynamic shape.
+  // NOTE: The func below was modified to adapt the dynamic shape.
   // Initialize the inference network, so that TensorRT layers can add to this
   // network.
   void InitNetwork();
diff --git a/python/paddle/incubate/distributed/fleet/parameter_server/pslib/__init__.py b/python/paddle/incubate/distributed/fleet/parameter_server/pslib/__init__.py
@@ -423,7 +423,7 @@ def set_file_num_one_shard(self, table_id, file_num):
 
     def save_persistables(self, executor, dirname, main_program=None, **kwargs):
         """
-        Save presistable parameters,
+        Save persistable parameters,
         when using fleet, it will save sparse and dense feature.
 
         Args:
diff --git a/python/paddle/static/quantization/post_training_quantization.py b/python/paddle/static/quantization/post_training_quantization.py
@@ -1653,7 +1653,7 @@ def quantize_weight_to_int(
 
     def convert_weight_to_fp16(self, save_model_dir):
         """
-        Convert all presistable vars from fp32 to fp16.
+        Convert all persistable vars from fp32 to fp16.
         Note that, this api only changes the data type of variables in
         __params__ file, and the __model__ file remains unchanged.
 

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ class GroupNormOpConverter : public OpConverter {`
`40`	`40`	`with_silu = PADDLE_GET_CONST(bool, op_desc.GetAttr("with_silu"));`
`41`	`41`	`}`
`42`	`42`
`43`		`- // get the presistable var's data`
	`43`	`+ // get the persistable var's data`
`44`	`44`	`auto GetWeight = [&](const std::string& var_name,`
`45`	`45`	`phi::DDim* dims) -> TensorRTEngine::Weight {`
`46`	`46`	`auto* temp_var = scope.FindVar(var_name);`