From 6f5669f9bf63985e4214889c94f6e97f26f8e3ab Mon Sep 17 00:00:00 2001
From: Zhou Wei <52485244+zhouwei25@users.noreply.github.com>
Date: Wed, 22 Apr 2020 14:10:27 +0800
Subject: [PATCH] Add note about the time cost and change HTTPS to HTTP to
 avoid unable to download(#24043)

---
 cmake/third_party.cmake     | 2 +-
 python/paddle/fluid/clip.py | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/cmake/third_party.cmake b/cmake/third_party.cmake
index 9c8a9e0af1..837babea02 100644
--- a/cmake/third_party.cmake
+++ b/cmake/third_party.cmake
@@ -210,7 +210,7 @@ list(APPEND third_party_deps extern_eigen3 extern_gflags extern_glog extern_boos
 list(APPEND third_party_deps extern_zlib extern_dlpack extern_warpctc extern_threadpool)
 
 # download file
-set(CUDAERROR_URL  "https://paddlepaddledeps.bj.bcebos.com/cudaErrorMessage.tar.gz" CACHE STRING "" FORCE)
+set(CUDAERROR_URL  "http://paddlepaddledeps.bj.bcebos.com/cudaErrorMessage.tar.gz" CACHE STRING "" FORCE)
 file_download_and_uncompress(${CUDAERROR_URL} "cudaerror")
 
 if(WITH_AMD_GPU)
diff --git a/python/paddle/fluid/clip.py b/python/paddle/fluid/clip.py
index 23a983d845..0b9abdf38d 100644
--- a/python/paddle/fluid/clip.py
+++ b/python/paddle/fluid/clip.py
@@ -843,6 +843,8 @@ def append_gradient_clip_ops(param_grads):
 
 
 # change wrong mapping relation between param & grad in clip op
+# Note: This function is sensitive to the time cost of the network with gradient clipping 
+# and should not be changed easily. If you must change, please test the time cost.
 def _correct_clip_op_role_var(params_grads, param_new_grad_name_dict):
     block_id_list = []
     if len(param_new_grad_name_dict) == 0:
-- 
GitLab