Fix some typos (accesor -> accessor) (#61758)

co63oc · web-flow · commit 95c16fe452ad · 2024-02-18T21:23:08.000+08:00
diff --git a/paddle/fluid/distributed/ps/service/brpc_ps_server.cc b/paddle/fluid/distributed/ps/service/brpc_ps_server.cc
@@ -314,7 +314,8 @@ int32_t BrpcPsService::PullDense(Table *table,
   uint32_t num = *(const uint32_t *)request.params(0).c_str();
 
   auto res_data = butil::get_object<std::vector<float>>();
-  res_data->resize(num * table->ValueAccesor()->GetAccessorInfo().select_size /
+  res_data->resize(num *
+                   table->GetValueAccessor()->GetAccessorInfo().select_size /
                    sizeof(float));
 
   TableContext table_context;
@@ -522,7 +523,7 @@ int32_t BrpcPsService::PullSparse(Table *table,
   CostTimer timer("pserver_server_pull_sparse");
   const uint32_t num =
       *(reinterpret_cast<const uint32_t *>(request.params(0).c_str()));
-  auto dim = table->ValueAccesor()->GetAccessorInfo().select_dim;
+  auto dim = table->GetValueAccessor()->GetAccessorInfo().select_dim;
 
   thread_local std::string req_buffer;
   req_buffer.reserve(req_buffer_size);
diff --git a/paddle/fluid/distributed/ps/table/memory_dense_table.cc b/paddle/fluid/distributed/ps/table/memory_dense_table.cc
@@ -224,9 +224,9 @@ int32_t MemoryDenseTable::Load(const std::string &path,
     VLOG(1) << "load dense table file list: " << ff;
   }
   size_t dim_num_per_file = _config.accessor().fea_dim() / file_list.size() + 1;
-  // param_dim_ in last node != _config.accesor().fea_dim() / _shard_num + 1
+  // param_dim_ in last node != _config.accessor().fea_dim() / _shard_num + 1
   size_t dim_num_per_shard =
-      _value_accesor->GetAccessorInfo().fea_dim / _shard_num + 1;
+      _value_accessor->GetAccessorInfo().fea_dim / _shard_num + 1;
   size_t start_dim_idx = dim_num_per_shard * _shard_idx;
   size_t start_file_idx = start_dim_idx / dim_num_per_file;
   size_t end_file_idx = (start_dim_idx + param_dim_) / dim_num_per_file;
@@ -238,9 +238,9 @@ int32_t MemoryDenseTable::Load(const std::string &path,
   int load_param = atoi(param.c_str());
   FsChannelConfig channel_config;
 
-  channel_config.converter = _value_accesor->Converter(load_param).converter;
+  channel_config.converter = _value_accessor->Converter(load_param).converter;
   channel_config.deconverter =
-      _value_accesor->Converter(load_param).deconverter;
+      _value_accessor->Converter(load_param).deconverter;
   bool is_read_failed = false;
   int err_no = 0;
   int retry_num = 0;
@@ -334,9 +334,9 @@ int32_t MemoryDenseTable::Save(const std::string &path,
         "%s/part-%03d", TableDir(path).c_str(), _shard_idx);
   }
   _afs_client.remove(channel_config.path);
-  channel_config.converter = _value_accesor->Converter(save_param).converter;
+  channel_config.converter = _value_accessor->Converter(save_param).converter;
   channel_config.deconverter =
-      _value_accesor->Converter(save_param).deconverter;
+      _value_accessor->Converter(save_param).deconverter;
 
   bool is_write_failed = false;
   std::vector<std::string> result_buffer_param;
diff --git a/paddle/fluid/distributed/ps/table/memory_sparse_table.cc b/paddle/fluid/distributed/ps/table/memory_sparse_table.cc
diff --git a/paddle/fluid/distributed/ps/table/ssd_sparse_table.cc b/paddle/fluid/distributed/ps/table/ssd_sparse_table.cc
diff --git a/paddle/fluid/distributed/ps/table/table.cc b/paddle/fluid/distributed/ps/table/table.cc
@@ -97,7 +97,7 @@ int32_t Table::InitializeAccessor() {
                << ", accessor_name:" << _config.accessor().accessor_class();
     return -1;
   }
-  _value_accesor.reset(accessor);
+  _value_accessor.reset(accessor);
   return 0;
 }
 
diff --git a/paddle/fluid/distributed/ps/table/table.h b/paddle/fluid/distributed/ps/table/table.h
@@ -151,8 +151,8 @@ class Table {
     return InitializeShard();
   }
 
-  inline std::shared_ptr<ValueAccessor> ValueAccesor() {
-    return _value_accesor;
+  inline std::shared_ptr<ValueAccessor> GetValueAccessor() {
+    return _value_accessor;
   }
 
   virtual void *GetShard(size_t shard_idx) = 0;
@@ -176,7 +176,7 @@ class Table {
   size_t _shard_num;  // table 分片总数
   TableParameter _config;
   float *_global_lr = nullptr;
-  std::shared_ptr<ValueAccessor> _value_accesor;
+  std::shared_ptr<ValueAccessor> _value_accessor;
   AfsClient _afs_client;
 };
 REGISTER_PSCORE_REGISTERER(Table);
diff --git a/paddle/fluid/distributed/ps/wrapper/fleet.cc b/paddle/fluid/distributed/ps/wrapper/fleet.cc
@@ -534,7 +534,7 @@ void FleetWrapper::PushSparseFromTensorWithLabelAsync(
     int fea_dim,
     uint64_t padding_id,
     bool scale_sparse,
-    const std::string& accesor,
+    const std::string& accessor,
     const std::string& click_name,
     platform::Place place,
     const std::vector<std::string>& input_names,
diff --git a/paddle/fluid/distributed/ps/wrapper/fleet.h b/paddle/fluid/distributed/ps/wrapper/fleet.h
@@ -184,7 +184,7 @@ class FleetWrapper {
       int fea_dim,
       uint64_t padding_id,
       bool scale_sparse,
-      const std::string& accesor,
+      const std::string& accessor,
       const std::string& click_name,
       platform::Place place,
       const std::vector<std::string>& input_names,
diff --git a/paddle/fluid/framework/fleet/fleet_wrapper.cc b/paddle/fluid/framework/fleet/fleet_wrapper.cc
@@ -1107,7 +1107,7 @@ void FleetWrapper::PushSparseFromTensorWithLabelAsync(
     int fea_dim,
     uint64_t padding_id,
     bool scale_sparse,
-    const std::string& accesor,
+    const std::string& accessor,
     const std::string& click_name,
     platform::Place place,
     const std::vector<std::string>& input_names,
@@ -1121,18 +1121,18 @@ void FleetWrapper::PushSparseFromTensorWithLabelAsync(
   int slot_offset = 0;
   int grad_dim = 0;
   // don't worry, user do not have to care about all these flags
-  if (accesor == "DownpourCtrAccessor" ||
-      accesor == "DownpourCtrDymfAccessor") {
+  if (accessor == "DownpourCtrAccessor" ||
+      accessor == "DownpourCtrDymfAccessor") {
     dump_slot = true;
     slot_offset = 1;
     grad_dim = fea_dim - 2;
     show_index = 1;
     click_index = 2;
-  } else if (accesor == "DownpourFeatureValueAccessor") {
+  } else if (accessor == "DownpourFeatureValueAccessor") {
     dump_slot = false;
     slot_offset = 0;
     grad_dim = fea_dim - 2;
-  } else if (accesor == "DownpourSparseValueAccessor") {
+  } else if (accessor == "DownpourSparseValueAccessor") {
     dump_slot = false;
     slot_offset = 0;
     grad_dim = fea_dim;
diff --git a/paddle/fluid/framework/fleet/fleet_wrapper.h b/paddle/fluid/framework/fleet/fleet_wrapper.h
@@ -253,7 +253,7 @@ class FleetWrapper {
       int fea_dim,
       uint64_t padding_id,
       bool scale_sparse,
-      const std::string& accesor,
+      const std::string& accessor,
       const std::string& click_name,
       platform::Place place,
       const std::vector<std::string>& input_names,
diff --git a/paddle/fluid/operators/pull_sparse_op.h b/paddle/fluid/operators/pull_sparse_op.h
@@ -43,7 +43,7 @@ void PushSparseFunctor(const framework::ExecutionContext& ctx) {
   auto inputs = ctx.MultiInput<phi::DenseTensor>("Ids");
   auto grads = ctx.MultiInput<phi::DenseTensor>(framework::GradVarName("Out"));
   uint32_t fea_dim = static_cast<uint32_t>(ctx.Attr<int>("EmbeddingDim"));
-  std::string accesor = ctx.Attr<std::string>("AccessorClass");
+  std::string accessor = ctx.Attr<std::string>("AccessorClass");
   bool scale_sparse = ctx.Attr<bool>("ScaleSparseGrad");
   uint64_t padding_id = static_cast<uint64_t>(ctx.Attr<int>("PaddingId"));
   const std::string& label_name = ctx.Attr<std::string>("CtrLabelName");
@@ -58,7 +58,7 @@ void PushSparseFunctor(const framework::ExecutionContext& ctx) {
                                                 fea_dim,
                                                 padding_id,
                                                 scale_sparse,
-                                                accesor,
+                                                accessor,
                                                 label_name,
                                                 ctx.GetPlace(),
                                                 input_names,

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ int32_t Table::InitializeAccessor() {`
`97`	`97`	`<< ", accessor_name:" << _config.accessor().accessor_class();`
`98`	`98`	`return -1;`
`99`	`99`	`}`
`100`		`- _value_accesor.reset(accessor);`
	`100`	`+ _value_accessor.reset(accessor);`
`101`	`101`	`return 0;`
`102`	`102`	`}`
`103`	`103`