From 6e964ad5cbce4c0c9a4f2365aaac0905e6ecf71a Mon Sep 17 00:00:00 2001
From: wanghaoshuang <wanghaoshuang@baidu.com>
Date: Thu, 7 Sep 2017 07:34:30 +0800
Subject: [PATCH] Fix issues

---
 paddle/operators/clip_op.cc | 4 ++--
 paddle/operators/clip_op.cu | 4 ++--
 paddle/operators/clip_op.h  | 8 ++++----
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/paddle/operators/clip_op.cc b/paddle/operators/clip_op.cc
index 8eea843889f..65bb0ba0165 100644
--- a/paddle/operators/clip_op.cc
+++ b/paddle/operators/clip_op.cc
@@ -26,8 +26,8 @@ class ClipOp : public framework::OperatorWithKernel {
  protected:
   void InferShape(const framework::InferShapeContext &ctx) const override {
     auto x_dims = ctx.Input<Tensor>("X")->dims();
-    auto max = GetAttr<float>("max");
-    auto min = GetAttr<float>("min");
+    auto max = Attr<float>("max");
+    auto min = Attr<float>("min");
     PADDLE_ENFORCE_LT(min, max, "max should be greater than min.");
     ctx.Output<Tensor>("Out")->Resize(x_dims);
   }
diff --git a/paddle/operators/clip_op.cu b/paddle/operators/clip_op.cu
index 51941deecec..7073fcb023f 100644
--- a/paddle/operators/clip_op.cu
+++ b/paddle/operators/clip_op.cu
@@ -34,8 +34,8 @@ template <typename T>
 class ClipGradientOpCUDAKernel : public framework::OpKernel {
  public:
   void Compute(const framework::ExecutionContext& context) const override {
-    auto max = context.op().GetAttr<float>("max");
-    auto min = context.op().GetAttr<float>("min");
+    auto max = context.op().Attr<float>("max");
+    auto min = context.op().Attr<float>("min");
     auto* d_out = context.Input<Tensor>(framework::GradVarName("Out"));
     auto* d_x = context.Output<Tensor>(framework::GradVarName("X"));
     auto* x = context.Output<Tensor>("X");
diff --git a/paddle/operators/clip_op.h b/paddle/operators/clip_op.h
index b9a2c61f72e..d596504bd83 100644
--- a/paddle/operators/clip_op.h
+++ b/paddle/operators/clip_op.h
@@ -30,8 +30,8 @@ template <typename Place, typename T>
 class ClipKernel : public framework::OpKernel {
  public:
   void Compute(const framework::ExecutionContext& context) const override {
-    auto max = context.op().GetAttr<float>("max");
-    auto min = context.op().GetAttr<float>("min");
+    auto max = context.op().Attr<float>("max");
+    auto min = context.op().Attr<float>("min");
     auto* x = context.Input<Tensor>("X");
     auto* out = context.Output<Tensor>("Out");
     out->mutable_data<T>(context.GetPlace());
@@ -46,8 +46,8 @@ template <typename T>
 class ClipGradKernel : public framework::OpKernel {
  public:
   void Compute(const framework::ExecutionContext& context) const override {
-    auto max = context.op().GetAttr<float>("max");
-    auto min = context.op().GetAttr<float>("min");
+    auto max = context.op().Attr<float>("max");
+    auto min = context.op().Attr<float>("min");
     auto* d_out = context.Input<Tensor>(framework::GradVarName("Out"));
     auto* d_x = context.Output<Tensor>(framework::GradVarName("X"));
     auto* x = context.Output<Tensor>("X");
-- 
GitLab