From b67d78e7ae57c635c7247f1eb36d507a30fa8fdf Mon Sep 17 00:00:00 2001
From: wangyanfei01 <wangyanfei01@baidu.com>
Date: Wed, 14 Dec 2016 18:31:37 +0800
Subject: [PATCH] fix bug: if test and sparse_remote_update can not co-exsit,
 crash trainer if necessary

---
 paddle/trainer/Trainer.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/paddle/trainer/Trainer.cpp b/paddle/trainer/Trainer.cpp
index 1eec2c432d2..3082b279dfc 100644
--- a/paddle/trainer/Trainer.cpp
+++ b/paddle/trainer/Trainer.cpp
@@ -222,6 +222,12 @@ void Trainer::init(const std::shared_ptr<TrainerConfigHelper>& config,
         DataProvider::create(config_->getTestDataConfig(), *config_, gpuData));
   }
   if (testDataProvider_) {
+    if (config_->getOptConfig().use_sparse_remote_updater()) {
+      LOG(FATAL) << "It's prohibited to set sparse_remote_update "
+                 << "in some layers if testing will be under going "
+                 << "in the middle of training. You can do testing "
+                 << "within separate process.";
+    }
     createTester();
   }
 
-- 
GitLab