From 090372f24baba42f001d2cbe0301fda5c2f5438f Mon Sep 17 00:00:00 2001 From: qijun Date: Thu, 19 Oct 2017 12:58:33 -0700 Subject: [PATCH] fix persistable bug --- paddle/operators/elementwise_op.h | 2 ++ python/paddle/v2/framework/framework.py | 2 +- python/paddle/v2/framework/layer_helper.py | 7 +++++-- .../paddle/v2/framework/tests/image/ranges.png | Bin 0 -> 6668 bytes .../paddle/v2/framework/tests/test_fit_a_line.py | 7 ++++++- 5 files changed, 14 insertions(+), 4 deletions(-) create mode 100644 python/paddle/v2/framework/tests/image/ranges.png diff --git a/paddle/operators/elementwise_op.h b/paddle/operators/elementwise_op.h index fce4b24a22f..b60cf0a985b 100644 --- a/paddle/operators/elementwise_op.h +++ b/paddle/operators/elementwise_op.h @@ -34,6 +34,8 @@ class ElementwiseOp : public framework::OperatorWithKernel { auto x_dim = ctx->GetInputDim("X"); auto y_dim = ctx->GetInputDim("Y"); + LOG(INFO) << x_dim; + LOG(INFO) << y_dim; PADDLE_ENFORCE_GE(x_dim.size(), y_dim.size(), "Rank of first input must >= rank of second input.") ctx->SetOutputDim("Out", x_dim); diff --git a/python/paddle/v2/framework/framework.py b/python/paddle/v2/framework/framework.py index 622e09fdde9..30ea5c99ddc 100644 --- a/python/paddle/v2/framework/framework.py +++ b/python/paddle/v2/framework/framework.py @@ -15,7 +15,7 @@ class Variable(object): shape=None, dtype=None, lod_level=None, - persistable=False, + persistable=None, **kwargs): self.block = block diff --git a/python/paddle/v2/framework/layer_helper.py b/python/paddle/v2/framework/layer_helper.py index 5cc4ffb626a..b4373493dc0 100644 --- a/python/paddle/v2/framework/layer_helper.py +++ b/python/paddle/v2/framework/layer_helper.py @@ -121,10 +121,13 @@ class LayerHelper(object): def create_tmp_variable(self, dtype): return self.program.current_block().create_var( - name=unique_name(".".join([self.name, 'tmp'])), dtype=dtype) + name=unique_name(".".join([self.name, 'tmp'])), + persistable=False, + dtype=dtype) def create_global_variable(self, *args, **kwargs): - return self.program.global_block().create_var(*args, **kwargs) + return self.program.global_block().create_var( + *args, persistable=False, **kwargs) def append_bias_op(self, input_var): size = list(input_var.shape[1:]) diff --git a/python/paddle/v2/framework/tests/image/ranges.png b/python/paddle/v2/framework/tests/image/ranges.png new file mode 100644 index 0000000000000000000000000000000000000000..5d86b12715f46afbafb7d50e2938e184219b5b95 GIT binary patch literal 6668 zcmdT}2T+sSp8pUHT( zh`bGwySwvt=I*?iy>DkWLx#zhK>Scqh9E~t zm@7(pUh%U7-gc;G|f@P>QX zJw%j}{o=(DOdEkuJ?i=MsrIb!^<`RGf{Z7gTIK-L!^Vi1L?tGgI&NyjUCgnl3q!~D znCVGrYrFC2(W9K3<&Hk%L>w>1b>cVY-|mimj&Pw)j*sVPCCPuM)4K};H>U+8Z!5A{ zF=U@VEMokMiJjZtFxs*yNnz-ky83G<1*eFy4J^kHM$@;o&9O9$9-o}F=HTQs64cLs z6)j?dH;l&4%?@pE(e+hT9}^=Y%y%&{8AV$fmOBlVRk}}GaU4Dj(-O87y*AT@Y<{;cyOfsu>gqtVG`F63euoMe5y3Dx#TwUzi+T4J5^D&WU zqz0L^otrbw5F|V=@~qa>mz2v!Z@r2f2jAu>Jz%22xUs*Q-)Gso*L{82*8k^X-FR71 zuYFj#aHGrH&7Vsx>V)r0OYYytFuQi{yh@PYTD5@X^f>mF*v3lOMPIviP28bRJVDA; z)uhrbf>%mv;LXW0pA~#YW@bZueNMlR*>5K}IQZh4GFue3jk#nz{64Df^77k%t7bnB z56{ZC*2|1ctj~F2!FDk3SXuAlpFe*tx2cQP7YlWgOwe5*@cM61hq@L=>-aEr;XEA@)gb`!9-RSWO;5J_ z&GAxGop~{*PMzu?naI4UsJKdycArZB@UKp25 zLw7e0u6WJ{PIct4u(NjuZY}XhNlDrE7F&M*@nh>#*a7Xx-u&Y3A_P$+4re9*Lt_Wk z%>}<=VmrnwDk|D;fgl33k=hU=vPS_;jJwa`@GZDp`{`3GHzT5r(b3V#RLedjqV!Lq z!f??>up}az+3>Uab2#hv%M$|kF8t59(K*%q{QQLr68`IMYpbg^Gd)GImz)yqFUCoG z&gDOUzMqem_uSdDYL1SM&q6{j$cZ2b9b3>Gr5JEP zPfsu1b+VaXOiYJFBEhi|`}5joI>2i@JR#RX$iK>9yt72Fg& zfBuI1REJIor*MPA@%{UggMtth6_um0+(^)NE_{Eb6Sp#zS19c$YE+Wlzj^SiX8u$J zK=rxKDPO?pjzj+nO#bUp{*MtQ^mIsNJ3;JAYV#q>M6ix&8JVFy12VzH8sO% zIt)lfA)aH{bI@B2Kt%_@zSL<*-rL9L8X-PDUQb&)`HV*FRf4RnY*bv_O|)L5?Cf)v z*4DU{9=KTs6nd~au$p8-&Sc!Z@7v9x_2}4(4g#2BA@$JXm-GVyD7^faF5&LoyEk6O zwr7vFqGC{LYHF^ZBeJU6V;>%n6ut6|{?r%hmMG<#$;&S-ZB)J9S%|JHKVN8VW23RQ zR;A!(x&ECy>9CSCPGO@XiJVB#MxH*yma>~$39oHys>=AFg7e4~#&!hJcK&}?<#-Kt z?E`_^8$-m{*xdepV~u!89U!mgO;)}ebhS6f?a3!Ew-AV3HS386(mv9s`= z%s%x5K0rMjoThOajUNNYsnTz)c(ja6CIhoII9QicIXE~npeu8^p)1LHWF$B^*sd;& z%c#;VN5rIZ%q#G9dln zpKp*t6vPQgNMIi@u@QheP)_{(`D!V?^I2mHG_Gx@Kn+j{ZhLE!tDFq0ncM2=>B;40 zMuLKG%%b?IeekbSP=EB+zi7m}7v&UH?16S}mf5#<=HWQFxC+*mCL66@=K!{{Ff;2Y zE8ho-9GjTPtf-J|AKii6HS3&Wt?-;5M0a@5Yn08Q=atew1It1fpj)F7n+T&)JfD`9 z76y7f5~>I%Mn2bh2;s^*%9Iu!xF@L>Q& zfX0Q&3kDox1{`r79v;|mHl)TR7aKI$l2i!gnf?@de5&_#);@08>_2PIDC)(NClwhO z7@j?Qc3xZ#LAJ34J7gXEB>@kboymdQTjO&{lFqlW;X9F<%Q+rHj7&)Kng3e_{lBe$ zP46gnC_GD48KLZbwR7-C#p9`xK* z0+70h8Aa&#BL7FLc9hua*40sGJ$h$70F&d6(7*mCs4^qjukh|n#`;SWP42WdzPeef zrdTP*!-o!KJCA-EmzRes6%`eM-XS%J-BYWx+OXO}eoZynwF;yG7#>^ZKs^Im5)_~9 z$0rAliQ7DX`}VEd(u9VJs%kD=b#Zq7LmY@G`FeSslaT0i_Wk_)m+`sq(9nXvbFM(m z_Q6$(5Hfrq8_pjClFk1YyP-d+C0ZgR2w!96yS!17uU6fHfB0WYs)oO* z%m-wFZ4^9pN?A)`vxFAWQEb_GtW*!bzj*^2~C z-5+8kJ&e2Lgn$2#86^zo(Z?dj70GZ&rarJMbb#2$-nfw(6?JHBZH-?@=(=@teCx_| z_xSjD#)n{bg=KaDA=ryxcAjWnhu&fpP;jl!I0VNgC$m7{XQZd=c_s1l^IyYYbih@X z!mgmcQe0fGwY4?ZteOJm2d1gM`J=pAxct*$C#?i&EZhlZ@+>s8^PraOB5~S(tq1KR z4{}w%JYSA4ea>fdrCZ`w4KmyNT1`D#C<2;h2P0!A*>j#nj)I=+mP9%9j#WM@C9ad2 zfPb|0m1!6dYigPpEow%3_;BYVAU-g)cram)!Yz9xh5|MmP`L%&Gu>6tZdX{9dEm6F z&5xO0Y2P0+m;Ff%eH4H4?c29YKu4i^4L%ha7Z(SOU!W4gnFA1yAAI|U*A|pIzMEu{ z(WQ>Tw1SfwqtSA}a=U-~qJ$#Kcw-DPF_8rG5tNiP0NF(D`uX{Z`z+h6%u;b#T1nBM z1WW@r-BC@E6tJGNwccHgpZM)1yDS}@#Qgkx4_zy8T^BsI8A$r?i;JyM{JOjt-7HOz z<4;FMM~N?9WYdQN&5j;D3ML(AVDN%d$RHUiUt3ov2v!4>%6hG6KuXfjv9lcViCS_i zQ6L|eKOJ|{TQwjK#FUqpgYRsRn^X%umQfpWSWr+g%%CqYxq2+lUz^1dz!kYe@m%KBuV0Vit^N4%+vkYW z^!~Yy9NoTG<`3-IS7q|)(jI0%-tFeewr)whrlr*##K2hMy<`Q}>T`Sh>r?FYUvsh& z+Y;q`c`=KNi|b2C+d@uGP5{V)^75u|1>f^3CG+~rRPfI@0gBm?OOEk={{9eBWM*o_ zO>NSLVk`~Um)o@jtDG=>mF{^^^em8Z{1CCg9uU&OrKuB+U2^>SIgrl@&JD12JH>xv zxg~H*w=LoD4)qujAPMSe!?YRaX=-_4qSpvBmqI|lbKu>(uaqxeCPqf4_4F7tCCHS& zx8pr?MgaDvx3`ytjja=ED{k3v$YF?Lvbny}{zX7 zXG7jVvBmvX7j_R$O#7`#CB?_jczxZftyck8huVsmn5^XFlR%pTE34er#`j4N(dgvS zVbi<)6ga)A;oJozPh8f| zi3eZ_<%2%ciF7%7zpH2M+FpR5Zm6pSa5M;?O}p;3??oFkuUo|N9p^%iNn8W8ir_6U ztuj(lQp&wqPJ$hl@cw=Wb_|!2lCoZ>u%+|0?jkf^fkBZn0p>ips{Wozz3J`kO_=*R zUgkz?^QD~hI0EMhjkB?+8O4|;(dYxYg#?5=n6?tzc5&M_Ubf>GuS3_8mMM+g<Hp`)V|-I_%TZb?~S#w$u4so5TG@O#DE&4^;#zp^YW${Te!HnNdQv) z?q)_tu~3eVQac=`jXX~ggm?tag`jv9^YVbH89ZAH!C_QWQ`4}*?%kzO%w4;8H`di< zDO3H4g}%*6((cy^j7r;}15Svt>TolV9?l2Q$>Ys?rn?IZ0pD!DeMyn_TDbKlyugH9 z28snrEKoJ4#^ayvy)?mp=FF!>O3LM6{CESAIOv(C8oCU41;$R-bHHP+P2DepI`W1c+OlaZBW z2a<^}w^JU>C{RQ0GNrrFtZ-(=9w=!Vcs8u=!rfs-6oMf!bBSkwUdS_iZACPFoe$?s zhQdTGwaePIm(vCkt*{HF4d;CO;pikw*mZrYq7Sx1$kpg?tf|@D9Y7vc>Y4RGNei-3 zEHnh*nj6Gd=r-{9E!=~b++K5*jnPu4Zt8;>gdHe_m=e?0S87-4OLs))3T;at$O;b+ z-^V3AR$=mEdPPa)$`zZYSP5zH#E;501kC91W~F$dX1rtyu!;<2ONToqS@0KGS!0w! z;E+P!F1M=jU@BcRCGLFQ54MrGu?l%F$l0bBVre&~Av4p5jOE#rC(qp`nQ1dK`Evtg zJGce0ubg*t;!?Fvo_@ihg9xaKeZM)dOR>(0oXhVn=dts;js@P@^u`j zY-nusO>VqLm+>&`sq!swjn0NheCGSk3uk^KV$5&$FUXD>PSYr|V_ql#UaEuX_hGB=B%+1Z$ zp*1?Pb<)UXYwy?tlF^Ljz<~p(^1A=v!T6Y7k!u$86b%L2M<4t}!CH1gpna#D3YN3b zSAJ4mT^(c=Ke`CQ@@k12