From 0130cfe15086d07ac4f08c187b07cabb61d1c2d0 Mon Sep 17 00:00:00 2001 From: Makoto YUI Date: Fri, 18 Oct 2013 22:21:57 +0900 Subject: [PATCH] Fixed score calculation w.r.t. bias values in predict() --- .../BinaryOnlineClassifierUDTF.java | 2 +- target/hivemall.jar | Bin 130410 -> 130408 bytes 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/hivemall/classifier/BinaryOnlineClassifierUDTF.java b/src/main/hivemall/classifier/BinaryOnlineClassifierUDTF.java index a6c65de7..58d5d334 100644 --- a/src/main/hivemall/classifier/BinaryOnlineClassifierUDTF.java +++ b/src/main/hivemall/classifier/BinaryOnlineClassifierUDTF.java @@ -200,7 +200,7 @@ protected float predict(final List features) { if(biasKey != null) { WeightValue biasWeight = weights.get(biasKey); if(biasWeight != null) { - score += (biasWeight.getValue() + bias); + score += (biasWeight.getValue() * bias); } } diff --git a/target/hivemall.jar b/target/hivemall.jar index 2bc1f04caa9eb179408fecd65313013f18550c96..be83f3c7eeed1348e6d99718a3d520dcd1a27c82 100644 GIT binary patch delta 2641 zcmY*ZdmxnQ8uxskB-aslMU84prL5h-8%a}C zM45)Pt%%yNTB$6OsTr4Xzh83S&TF*0=Y0Qs&+qpq! zywp0^#U_OkHdCS0Tt6tV+tK4ke!fag=eAvTAC$+wzWI(}v{K9JH~Bp&uWx%UbZjV+ zV~2lk?0VEo?X?(Q`SZKO>79NTcf9|lQk=1;Wy7xBmQVLCxxj7qc|{tzN2P)k!uV9nQ3(nOycJ1#OgDWyMg02;Ajtzf#uOs-iF5fBd zfb4<%*d4sVfHQ8V4nG~1&=TU3nDx;c+!y3@FwTV~t{Lk0x|hog6y1^a^WQnCf2Q{P z&PN<)(d0JYi`Uh*8h0G!Y;`Su71gHoo7c%-J=(u{S3VH&k9JWdBf{oPt^fGSqOpGY ze2&dP#FMNW5ixP9x%+veY&oCY0N&`RQodg(Z?$%L#%`TswgtiQM=LHUTg}U4->DTx z=bVqKiu8G?QW4b}mLiut`7ks|F4Zkm$~8*Em^pD-XBMDDp{!vR{w{YtVya#Zhl zG1*{{S$#IAt#u~4=ikZ=K@(c_lCwRg&IRt80eXd^uX_~3@3$;TC{BV2v89s9e^wOS zh^n|c2p>K8V_lTJyxptq%5?;uxnVmW_CHKM!O5+;E#U6k5i-4@%zMq+Pfkxp<%9(% ze(Grs-CvTpM(topw#P5&cxjXam))laE~Dkuy*6t8wIn%g@416pvmUfl$};CQdKcDO z8dmeHA0=oiSH#3s#wb46)0^Tns-{x;;s{go z&7HdDwIWgcX8G|+)6Y@M(yyv(e@Ko0-phclHdTD?gnnLm5%ry!X=rpW%Xep6ev)0u znk%{HAwH{B_(eGp+1C7`O)P0Db{i|4r0-CuE>tm zS^Jpe*>99NL4VU&;munvdYck%#6$Eexh6+qZEsgp+3lh2_==Hm*s$S-cZBY}zILyg zjQ1NQEvM5%bu5p_Z@z0i;+J4wU%ieVX+jyAS@B@F(CKlmt8pYblx*i2$SQew>&N1d zSa;4{Rc~9FV_qNF4h|U;XA|bPGDC|lZGRqW^u#1{A~9SwJ9Dr$hfC$V)`=D3&St3f z3d~Gi^c}Je-WTJ*%Z|6AP1_9VsqS>~Da?25PPW>6i_3SrlwL-w47L9>9b?N$y053; zIdmiYFRseMccZcvI! zg>an6I5bOfdk-0+xDP-loWs9hDvB3^CR+3nsPF<=d;~V(a9%1e{|Fn1vl-ZSK@1UC z68@Rkc1KmBS^ZsXujlga%As{)phFd6i{WcRn~yk6K;v%bFF^yCC(H$NM6f`6mY^X2 zCVbF(bg~IN!5h75f~~L|EolZ%Vt*mGv{DbnHG>^N6wlh+^~JjA=N7EeY{9B2WZ42f zMD25F+`SgCA-*Y-+H$SnKwK)9YAVZBkld{1bSqGa^h!)xq1UbWF1=OKU|k6~61-|@ z@W*P+xdU!WaPZ={(twW=uqWQtV0$UDYXb|IM!{{kINL_TqNIh(|D=UfJ75=4FZt76#x-k`Luq69N{>76DXf7%J@e6U2UrmEMlfN{ zO`Xy}2RiXRfBb~;>N!P2Csv3jFky(OU0^}FP01kW0Ud7ilm)ud1$2@%Gshrt7kCgt zNa_Jy>|Ka_yK(aEB*s>#q#GO|0DbNTf1-8)roKX#dT=@?ML_>lge6GWgHv6`C`SLz+tOiDigAoy_j#T@wtVkUfBCTe)7Y`)|sr11v&_)OPz>9dQi>muE_`MGg zC){8TWE+4U+B6PIh}jP`Fhe2z;ER*I>W7`Avo#==HO|{Tf+^MjP|4M6q|{P$Z2;Dh zFKne~F`Bjo)!8T|l-2_jRjSPYKHnkBLD)#XvIo>_4^(tPjN@($;^VYX$shz1!Rt`z zAgo1YB2Y(1hJZ#~b(&*d4}ldiy(L|%!f+JbS`GORq@g=85 zahKnomQrRYZ4@_P|0mqj%jnc6@FD$Tz62j0gJno_4A;lG@HfRI|A_w^si6KbysF2M z-Z(G`i$tV9jtlji0QsZ|P(aD!;7;5_|Hs-9t5W_V0tvU&zW@zg8N&t4OyK3am+=>^ zh-@bDN|k2LF;SEFOI3d`2VPF%@>b;odRhRB5j!6i&W@ad+@^3rJj9v;Z?d!qkhlmh z8=ejYNoGCDNz>p#GRpvkl;JHoIE}w8SCldhn}I}a_(dwz09sLlecm%bkqfE^Jof(r DN;IR% delta 2688 zcmY+Edpy*67ss97FGeBP!nkx%WGaLqmzAlMv}F=X61GSeVZ%~1iFIkoWz<(Kn-#Y8 zglJT1-Lh%c){-o_U&bx>$t@=K`5L?XdY=Ezd7sbup7T5BynX|7l)*WQBy&HxOhH6k zTwG**WPzl>6m8c68TPZ9l}P73tazkrCLzPh&9$j>eQh0JP@+1(3H5K})z;jTVqU9} zrG-BgcXhAkn@bIgKAS!%xnH;Y>t7;d$b)93ksJC?METU(m+g}e5uM>VOR9CeZGOu5 z$=KF|!%w~^wMLz-X8KF4#%l*Vd+rxwDD9Za#KRMLJ<;C_=1wU(i+z(e#Bt1E=5?Fv zvWioBIXW+nHf=q^wiCayeBWm08TX`!tbmk*cdP3@TzG0L_cBudOmTwyo`JduqtK-B z-$v=odx8$fGVeyOsM|hoUp`hjFk{E5({vQm8lh@cH(#HfAGA(YXndx2Unxq)k#>a1 z_qrcc$K;EKte~BkI(FDOGrjvQlYghkib`X($2OQWMqamNWLYz^>RpayI$yAI4ez@u zb;nA>@}3Kah-FBa2eN7oPf{V0|0euE#ZZ|&e_cSegVG)@>Wb&6|WX^Nt2kMnRA4*tz zS?MKV`Md*Cxn}+$L3A?rh(Yb8jbv{A>GY{2GWWObnr+RszZbZKzutB5e1f*6gITT^ z<&;&=xzcwz0hWClci$Ou>+^3uGUS>&t-VZ_IOV6Lr7h>PTECEK^uUbb$*L-jG`!nW z8m#DQE*UPGnsZGyep6$tNE>N1klAUN-opB8;p(VIMwPdmr%~O?bt&PDsYGt<>4c#0 zxL*&l)8H`EaLw;7sU>x^SFxF9&J}0KzoT9KhWnogUXYBO-vO`s!C-N>^Yj!T@^P<1}bwYo6 z2g~#Nyo+{4O{RiRb^SVS(go8d9j@hcyfU}+cG$3rGIzo8k{WlB)s36`3IZd7^(w9?_o>DOHK+i}NU367dYnD@2UE@FzCUX~ z@s_AQiaDfQdEVq~O2?KYn=LnIBkWd5xxt-HuQuP_kd?x3eNlWj`8!{xP8GW_+M;On ztJ#If<2%b6PtW$tE4QU`2l^sHVyTB>-p|m7C2oH)`e%8ddm?#Wn)gEgr=ogzToDo) z^Df8l>WD|Kb5_`h)e3g6n3O=vFRYczvQT`NWLK=MRbZAVI$g5Kbf3h;`|=IxPqN=0 zyWM6&3>=PW-;+&joW0eMX#gem6(KGa&abywU1<35@!`&}49@1?R%Ye!yaIN_-_iWj zg0@Cc^=38RyKh zw+l_LhT8829eG{r-sonUuSH2i9uKIu47ywxZ}KxQAGM0h_e`t5y0uX3i0_sSab76E zNjos=o|C*;54e1r%zcEsoNAJ1X=xGkuT<&N%v&u5{X1UIKJphm6qF?G zGWEO0<4w9I`~EEGl=J>Eez5=KM-!KC<)d4*zwhz0ed{X1+)oid>R)(qD4fB3!eD}7 z6tO?Wcz z**|95pq~bCwnqn0!2@0GfB<554(68ZA9BPHvjrrP3?B;~48uyOL0+A(k5GAtLc2hn zefXgo8tufdZYx{@X9|@Ufedo*!q4qQAzipp$0D|Gkp>#?!a~g_Y|AGbkYzWvy`HoA zm1|LKH<%L3N*8UmMX4%!%ES2zJlywuxlq95feF!AiS4b6K^oyyh3)l7tOrct24eJp z6OmOd%RLxthHt}O6bKgYMn6F+6!hx^E+&>LEHO44;Ijw zKKyZJEo_GtP1M{6G&qVx`@tTR(6N3v48bU`A1~nrkG*bE5wQkfJ<=TjW1^||A7DqE z9RJ6rAo?iY0ObSz63Qf|yOt&$hyf&{#1G&`bkAbG5h)FV9$_s&ek7=}uL$&(gxP~w zcoJ0%!Wpt10ELNy3XaO5V?&@v9wCKf8!8yCeXe zkN`E*JPb549(y?wps{pbrXxTlePscCCkvV=bOg^PSRP4@!Y1@w9%O`X@v#Q-kmM*h zfeP{*g+pY8DxgFSJe0aoFeU>vgx+>EH-=9IU$i>tA)hfgPRechTh}*+JKSvu$kh-( zZ99$|F*Xu9W`KswfQqX7af!P)LMWS~ z@(Da$9V=LaCiy@oym$N$ar+2{L>lA&$ucy~019gOh!?wnU;;dexdTfQ)&%T;pHbNa zI1{pG|78?pG6{y{Ek8g7et6rsz6{Ek#0%~BLw-}Bh?Kqtg~f(t(Ci|%@L$3{{-BIf zr!aPy5-!nq3LJ^|tfgpq1c;;lDKIDMuPqVtY1m1`NBkRBLiy8p8*WjHL>UFnz*?j* z1AB?En12bA5J_Be3=%;OnT_IVr|nqhvo+HECsXp9P%@TNDa~F@R3#C2c%bkzmqiwbaGXJ(9=Y1bGSJ=2e}Dw zb6y-=T{wc>0$e=es!;7iiJ3Xo3~{vhwqcoBP1%4gV5 kioOA~_6^=;@#n=KP7^FctLAZzB+{D)eX(