From f32859b1e0d745b59518db4c314b77604ab91933 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89tienne=20T=C3=A9treault-Pinard?= Date: Mon, 20 Mar 2017 11:04:09 -0400 Subject: [PATCH 1/2] make sure histograms get a round of bar array-to-calcdata - require to make colorscale work on histogram traces --- src/traces/histogram/calc.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/traces/histogram/calc.js b/src/traces/histogram/calc.js index 9b079438b98..95f97930e72 100644 --- a/src/traces/histogram/calc.js +++ b/src/traces/histogram/calc.js @@ -14,6 +14,7 @@ var isNumeric = require('fast-isnumeric'); var Lib = require('../../lib'); var Axes = require('../../plots/cartesian/axes'); +var arraysToCalcdata = require('../bar/arrays_to_calcdata'); var binFunctions = require('./bin_functions'); var normFunctions = require('./norm_functions'); var doAvg = require('./average'); @@ -168,6 +169,8 @@ module.exports = function calc(gd, trace) { } } + arraysToCalcdata(cd, trace); + return cd; }; From 9a07ce197177fc2b529e892049542a0d4ee649d5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89tienne=20T=C3=A9treault-Pinard?= Date: Mon, 20 Mar 2017 11:04:20 -0400 Subject: [PATCH 2/2] add histogram colorscale mock --- test/image/baselines/histogram_colorscale.png | Bin 0 -> 15295 bytes test/image/mocks/histogram_colorscale.json | 23 ++++++++++++++++++ 2 files changed, 23 insertions(+) create mode 100644 test/image/baselines/histogram_colorscale.png create mode 100644 test/image/mocks/histogram_colorscale.json diff --git a/test/image/baselines/histogram_colorscale.png b/test/image/baselines/histogram_colorscale.png new file mode 100644 index 0000000000000000000000000000000000000000..c83af4bd5e24df4510aa09cf4286b7267988e29a GIT binary patch literal 15295 zcmeHOX;f3^)&_(EA~IN|iV>_uyhT83HDZ8ZsZuAfdKCdfu&5{yqNOI0kc2^{l>)U^ zphbw*fm#GaAYsTrM4%c35g|y(00<7@kT7K+)AxoetoD9C?)T%ab?^FCen8;loOkbc z+RyX6d!O(21_#;BUpC*w#Kd;T7oYDlF)?G9n3zsly$7y{!gL}N6IYWRpKtv;ArcZ$ zTBT=DvN47A1LevVhn6+)?E&B3+q9Fr93O!XkBGB~bHBG?^sCDb->yMU`WLN_x89nv zJTUO)vwTy^Ip>=vS2^;({^W@6gX}%UUmy?muRP>h*!b;WtwNh{6+woTO^`W~DH6p$ zr?PgFmQ|)uPY_6xWTBIjlLQC#R+p#(Q%p?F%&qKQ1EPL4lbB0Chf699|IJ*^HA~}V z{^L(@ZQV&z4&u#J(&m5M5ANVdVDIj~YT;}{R=dLkWWh+b9PpHx)R)(k#<3|&bSho^JbhfT)4c%o^d&{JY&_YC-w~CtPzvc9X3|N z9p$`6hgnaoBBirNQW|brm`iW+B>i)y%**k!MgpRqnWf>MNv2NCnM&g!W{nuTYn)x< zVm9nE&9I46->QWignQ| zTks)JPershD=o+I9gxT)qx4#ZNWF4VJphqq=nFwEq$-lK(Ip{h#o43$s*{(+s2c@2 z4&Pd7UQu-Af7HAii=tvps;WZXOc<-N6#DaN4q3r1{oZt-4lS0OeND54b3z>kJ zXx+fi(JOPtbJe;r|#2*xI{mlor;ESrsrP2PrZn7HnCpHQuo%Kk>u)M>~Jr9-nM!0dmrYi zqALL)ba>v7ZaE~wRV@85z;$a>E=SFjlFCWbozw((YVy|zamu{jEV!^|1wovO8*6ex zW@uVBw8w{{&K!tflSvP@EZ>^apno51Tc+LI-f#VcKxK;6QZ^fPcIXN;Eh(q8jlwa# z&lBq2+!l!XNpi~-#E}XgrZ@26_XdiVrw{gPKe9e`o}DIn#3p}rfY`!fp}#U}i>WDx zAFh60?BBJdgQ!!zLS}qDh7vdBY#HgPxB(+gc0hM4+hQ|dxSr8=Px40(TtJ9ik&5Ay z^rJ4D(g*lduYPu;d-HTssY8K~^kCFN7l!F*hNVmLDb)>|OzE`0M%Kusz!F>6V|ZmKC^sVEq@pjeeVo2Y8JN<*^>!to z$%A?0<{b6D*y(U{E5=V|l?#OJ$-^6n%E?m#Z!T|Y zo3;H31HIjdFGHOw$n--HV;IodG zC@Xj*?U@pSCq4-P4x+onz>s-EbmXVq=LNX|0RgfyQ)x$#3#xfEET*t4cys@`jk}e9 z{661GctFrPA=+gitOklZLyGx;36mVyEiM!W=EeP6nBJ;8%r|?m7f%*=IoUIYJbFq% zd}I!MtiLDrL~|lh0kYA>d3JVQt^~!s28d6mXL{BBhUu%RC?pJgmtNjG5-bn@;-LUekn1n*;6z*(Qf`YOdP)BKjKy!;?gB!8V7$%K!JgqZBFLC8OsM;D zD<|TCko21vy1ilZH4yYqoh<7EsG^2GUTuLrpQQgejV>ztYM<=jZ-~mB0S3^be;zPr za_4&%v!1kxQ$SxDZqHD*uY)PL7hTJr7 z_^}oMfdHPsoPKjdVPhFXzaXk=8enVxaa>Z0k7e1x?&f4dSCs>eQs}eX$1ev zAlMW`!vBk~Qq2l0(@^NKjR8c6Gw^3P;h51ynB7q{j`~pu`93(LK|z4SpKv;SievMh z^yiqO7iD#Cf2WbP)A?0zt^mmMsJO5Z{h6%n=U6xYHH6TwY~xtx#HbBd?1BRftyGtj zN?{bu7-kIC2adB|S)}7TZ|{L_7bNs*VZs<`vO}xx`&F|2qUk%Qy(G8y$g-y#0VS7D zvG}^PcWv4=G@#A&trlP9Ao*g_FA!s*E$&qjaTGZ7$er4^DZb|^ZltFp8;BfScUVky z<3dC{eR@Eoh*_RW)yYG062st&8R^VzY*FzS-dk*Mc(z?Axl>%!TgH_4 z;$C0RSXeiH799aVk1~s_jx$Ol8X$;wI)1P@3%cN(aKaMy&EDO@Z@vRc_K4K~?=oEH zl?C&Kj`Gn!3oN{lV+u~&D$^7kOP9mC*AE6_&bW}3{X^_*pW%F39Ip(=^`uu3M{8qL z5zd;W`wbWi|6k@5VH~2Asi2jVvyxP(KA?=McX>0bJU=7=$sN}M!yiVdM=JR%GnPB< zhoivBmZ57)7^wJTfob&bS&}FYzv=jiy$0QRjhF}p15Ilk`sr#yrWld4m{E9YTXY&7 zOphDDv~z;%2f)Wpmh>=(|LWjIH$urvb=zVFDdBdrL1d3UDMbKT>3+B)U+kGC^*t!V zJy@J&E&Pzsl51rpbi`#G1eEP#6)VvKOO5FINnHA)6cH+u*Z#cB(79STeH1kF&LnH% z(3~f*h!0%}yLw>ZZmx4C5A2`w(H66w;Nxc|0Dt02%ACmq zd#qbFhW3BNpKHg;-oa} zeB1(}C8xf*%UZbcvr=V?x%8s<>oA@nPczp|klF(LB8v=u_Il{nbLt0dz%Y2e4x=w- zM3;ulXhvs#9!QT-#i&)!FH&~P%9Lq_bdMt=P-ndCrz;9Xn&avQOaG@U&c3{adwD5{<{3e&3S8*((&v`~ zI5t5cz@sBT5@;NWcyse|@6wp773xGuH_DNp^5*ZRoPx(N|J||+*89Q1-JYR^E~^QR z=kA~bWX&}(OBo`)PQiauOce@MZFm}-(QgZNM_@5%)#K1!w>)2XWV`P~E3WH6X!VqY zhb*rWXE9U$De%XS^_)$E59j%L5~2q{b_$z3=nR1pk~rSs#3%c(hP>8K`-4xy`%VQS zWDWDEE9*{n-o|zAfkV25(#l#3?1)8c(Gtdsfr<%pY^yghTy{LOrEL*oJ$vLN$9WO; zn2bD{NR1!|Dg2~o+Y^|H?S6FTAWV9qoCL*DS0;@=a;MumZ>oYJa8);Q{ObzObcaFr z8W-}lO95dptptRDMi}wsI8Si@YNq)i7yznvB!*9CYoVc>Pkpto1X3~{0abPOTDxDB zE$A?u>4NGNw&9Fjfa7DJY{cjZa0<=@}KcRLg7uW8mSc_Clx`_J!^+K?CfoDw=;)bq-(hR~IG?uIy zw9RBMg@i;YCKOGY8n3U()Q>#d<&D}dBdZ}E7~LA0vLi#UQYZb(b%BvJ_vrLhV$z+D z5?R^eC#T7Gf*ly%SB|#v92h=r-d%r7Q{4KoRkZ~y&b5HA=x$dV`O81;T_b5Tl362z zH!7R|tegWWS1fs=0Gj8b3?Kr2(|ajs#S2>ei)Vo1{2f1xrK(iIvOs6wT8h9H^D zi9%^SmeAMuSg1ju8HI`+l#;lGjt39r|gQ~|> z?+^C5B{v{q@~)It_MsVD%V$4X;|dbVDT){EWEx4$7e(s6XzZj2wWD}?XE36)@YDlq zQ{0WYw=BZI5s2M0yzp$yOnEL*q!tROTMWn9vn=B{^oP&f?I1l~Z^^)tBoR=$izZ6_ zN09%0&GWWG1RoSJ?;TU9g)+?lq>3l=!f$G)>rBhcs(i z>~vx~+Ma?<)o67!I^qepu4`XZOEwI;=uvh^_ueIuUMniT|G+Gh{ncbKWx5zjwOnxm zIT?t2PM&~MDz($)`sw*1f-qD)p2zOf4S4H*)k@g9IAW7WWQ9ibV1J0;8R43AUu_Fg zHzq&toqXL>SB^9<0;Rgb^8zypXr$9$uWI+Ew95?VN^MBJW(av=(49QszNYkAgQ2`k zs9-6o2eT1nT2KI-Ax3dv+I9~uB+_AsPvRmwvynRG5b}@VtBGwV^U&n8h=Ffc9n=g@ zFoXm2$>nFBf63MQf=oVU%sFX^NJbDyBHGIhU>BLJt>(ESC&yU_t}Vrbzt+Qbg`|A|JVJR-*68)O-3$856mbXqAhu-(|kA z3mx{>)&l#nBGyHYU{4^pE+XY67uBVqZPMCTMS$PZtC7ldvd>B)M7{pg9XT{m<9&cD0Ys3=Wq(?AM$+w2hU6pW z3&#%XD{@G8J%8n){lXx@Or=a)nQR!&K3nbWJIMFcJ^GEJdqrVs+C~S0 z_L@t-M_+Ive8Z6x_IRF}ep5{6dez_6_T*@L#zixyh-m#3TD?$uk-aJn-{drTSK`TX zW-GNqDMhd5`>RJbbPvTO@hNILB|?WQLaVNz!*m~9M3%V8K(P$X#;fKUDtWAl%l=ID z`bG{I%kYk!ecNSa50@Prbj;LOD7(+@;T@cg47JBvCu=2Ww9d9WQ~Jd z7lG;#kEGFo44qR)eS)BF2zg)>D7UbVg1YWb5Ny839~na&cIpRn-rY%A!ge_3uupfl zs9_*xE2v;E_o9p(9TO