Created by: reyoung

This issue used to record which variables are removed and will be removed. Related issue #849 (closed)

• FLAGS_cudnn_dir in hl_dso_loader.cc @gangliao
• FLAGS_cuda_dir in hl_dso_loader.cc @gangliao
• FLAGS_warpctc_dir in hl_dso_loader.cc @gangliao
• FLAGS_memory_threshold_on_load_data
  • ProtoDataProvider.cpp, Should we just removed ProtoDataProvider? because it is not useful. Or just give a deprecated mark.
• FLAGS_allow_only_one_model_on_one_gpu in MultiGradientMachine.cpp maybe it should be in ModelConfig
• diy_beam_search_prob_so in RecurrentGradientMachine.cpp
• log_error_clipping in Layer.cpp
• rnn_use_batch in RecurrentLayer.cpp
• allow_inefficient_sparse_update in SparseRowMatrix.cpp https://github.com/PaddlePaddle/Paddle/issues/1238
• pool_limit_size in Storage.cpp
• log_clipping in FirstOrderOptimizer.cpp
• enable_grad_share in Parameter.cpp
• grad_share_block_num in Parameter.cpp
• small_message in LightNetwork.cpp
• sock_send_buf_size in LightNetwork.cpp
• sock_recv_buf_size in LightNetwork.cpp
• pservers in ParameterClient2.cpp
• parallel_thread_num in ParameterClient2.cpp
• pserver_num_threads in ParameterServer2.cpp
• async_lagged_ratio_min in ParameterServer2.cpp
• async_lagged_ratio_default in ParameterServer2.cpp
• Flags in SparseParameterDistribution
  • It seems that they are not very useful
• Flags.h

DECLARE_bool(parallel_nn);
DECLARE_int32(async_count);
DECLARE_int32(port);
DECLARE_int32(data_server_port);
DECLARE_bool(use_gpu);
DECLARE_int32(gpu_id);
DECLARE_int32(trainer_count);
DECLARE_int32(ports_num);
DECLARE_int32(ports_num_for_sparse);
DECLARE_string(nics);
DECLARE_string(rdma_tcp);
DECLARE_int32(trainer_id);
DECLARE_int32(num_gradient_servers);
DECLARE_string(comment);
DECLARE_string(load_missing_parameter_strategy);
DECLARE_int32(log_period);
DECLARE_int32(log_period_server);
DECLARE_double(checkgrad_eps);
DECLARE_int32(enable_parallel_vector);
DECLARE_bool(loadsave_parameters_in_pserver);
DECLARE_int32(beam_size);
DECLARE_bool(show_layer_stat);
DECLARE_string(predict_file);
DECLARE_bool(prev_batch_state);
DECLARE_string(init_model_path);

I just go through Paddle code quickly, it seems the Flags.h should be our first thing to removed.

Or maybe we should think a strategy about how to remove them.

• 我们是不是应该在我们的Protobuf里面增加一些字段，这些字段就是这些flags，然后在main里面把这些flags直接set到protobuf里面。
• 亦或是，我们先一个一个删掉一些flags？