From ec5e150e001b983ee1b44aca22531001ce999e9f Mon Sep 17 00:00:00 2001 From: Andrzej Novak Date: Mon, 3 Feb 2025 16:30:21 +0100 Subject: [PATCH] fix: update test for new mpl behaviour (#542) * fix: adjust tests for mpl * fix: adjust tests for mpl, ruff * fix: version parsin --- tests/baseline/test_log.png | Bin 8724 -> 8811 bytes tests/baseline/test_log_mpl39.png | Bin 0 -> 8724 bytes tests/test_basic.py | 21 +++++++++++++++++++++ 3 files changed, 21 insertions(+) create mode 100644 tests/baseline/test_log_mpl39.png diff --git a/tests/baseline/test_log.png b/tests/baseline/test_log.png index 4a267e5726c6ca952cb7dd1a7153dfa3260418b4..9c7b1e913d8f15d3de73718875b217171e1901cc 100644 GIT binary patch literal 8811 zcmeHMc~nzZ8ovRKfU-!f6_v$h2J|SKLk$Q?u$4in0dYb0MUGmQu!^8b0MUb0ESWh1 zafljPDk2~V$WoRNi#7u>prSz71w+sT5(r6T37H$HYsWbq+c}+j@<(2h_wKvzzW2WG z`z_y@T`o@A3s)=z0BG;nzHK)Es>6zp#satpoAdAq{L+lv?imAsZlL0W$irL>24Ich zj%{1_#$O!iX$e{B*~@smwkt~Sq_1k(qH8OK)iLTP{O(?f=U!U%ohkO@&;o+h3EHVb zJSzJZ6Opkxcc4UUK7|uw4sY7gXzNeDtRCW~t%pg&84~yYSsYg9SG)*bb=M=m5d{1O zCv@OOPi+g_sx5|#1B)H9RRM6AKY&mkP(G4BYB1=8=+fI`^u?zle3V3z{$(~tV}b!WD=M<_;Mh;p3mmUNT;;(&paz;z9l zVa?ODVjFFOtwnMLBX!koA0MCe)YQ~yJdcG0>m0LPdk>VXi|DPrhFSn-8rf`5TRXcA z#6EHbE|jF2Q;Qdu8W@hY8x6W0lRvH?h#E-cjHGoc#DO8vFV~>`{(UXv3s&|;mlzf%eDfr^Y3!j3V27#%JZHzH|b=js_@3^ zuN6K`s%$G)tS2aHtQ@k$d-G$zay9{^0Y!}nmRbx=Qr zb*xXi^eP+Ij;!QNh`Tz2943cO8;6pFd9n#vC1?1aL=%Kjy}M1rOFYsma7ZUumz-1p z=r=M)>{Pr;E|*IY(FBgilj4f~GIJVaok8|qu2hej7BRfu;Apyqr6GeH9!@GZU@W`favGMZ4mTP- z{bTfBtO1-XcW<-Bqi24W)lt!jbzpS$GKed0MlH2_X zRy$W=;|tq)(Zm>U{5dC{Og$80IN_oLwJK1*$x=&^gZ0l#ARaTH@+!OZA-hh(C|<(b z6}VyP#2HEpmoqa%i0PHe+}Fr@3do${$i#_3e*RV0LYN{#JjBtg-1XSjFj*8yN6Nha zjlVIR4;Phc2j8U8v^f!GM03t-@g5D9>4Ey2 zxdo5%(|1FZqRKo3n3=2H+5)6}7IQj*nACIQYi_;&<+yg9*!?gi*}v^}ubQyF3<)!F zJ0qI|lKNkK-0kPvjaw>gb&%}Gl7ztyN-l#Ku%z7@8=N&1Cm*p$oMa9&P`0*X zf7+5PUdfmnx4bL?;={45;Et}YuBMEY^<9G=^-=Q|!?zxZr$E$rr8_1!Vq4P%wCPdT zN|vY|UgM{+7Gx@GetBt%iLoS$Gtd!80!nuH`}39$Q`7b*miQGj#Vp*2mC56$=VhBH zESPD+VK$|L(W{18GQ0$-_4)FYK!}^fJHh=--?!!M#DuP8W?e9zj}`s}ikdfx(v|ex z=H}*2PoD-K8yj#OaRaN8E3cV^qp4((NTi_QYc4qn=CQaQR=lQZ-AWF{IH3@=V5a$P z`P)H3Q|mNmcV@p7PIPyLyw%o8S|>3HSoy(oTP2kKH&Cji@epwSoufR*GeGQ{yt>k? zZJcRn=v=*itW^z)&!0)-FJ&U*!~&Fx_mfZJK=xUMYjym%mtSqX8~$VPRX?;nv>SG% z1a>GqsHhd-`J*?}w@lPqcib~gN2(CUDB^(;bbG7mH0P?I{yJ*GHof5`l^z?L=20G; zvfI)v^>nA@v%`ksbY|&hqf3y~7G_9!rvY>Z=RK7MZ})jX&XwrH`y?c9c$cL6<2&mR zctFj8IzKXx0=)`*F9F7*23L~lKRSG`LKD~<6kD5wzb0Av6>wbi)D*1f*>fJDQ~b@@ zNmBhIBLui$`LhxYwXDxpYSkpcueqfK$tCusP5^;l-A1R=)8&l#o0rigN2xpr4CAfR zzBVIOkR<=?RRtER!(nJ<)BaB>teIv6IHlmDAGTUj(13;&Dc@){+^`i)(*~>+;dgGs zTm(CJRZwzx)jwH$C(u!%$=X8q0#=ESdFZ@t&^>^Tg48cJ7=kQh`=(XNzdy10pEjD5 zprr(@-?*tc7gEn_!rn}7u5m3s<`v2Ig_BYr>d1dnE7>l64dmk#t(NZLW zE>mP*>YZ1v9bG{*I(gPH=PIk+=7CF2NkgXf7=0j zW2H#Wh3j>`rF%x9IOcE;+M?0sWo&+#oCrN*UUjLDIkT~`QFW&cZcZ0N;dz)p1)oSX zoboaX?=F!Le&AQ~v9X@7^z1}cGA&3Vku-_8El!x`#>O+I_w`;X>kHzT*ntE|uGWhQ z8NJqqy(X~Wz3RWAzi74Q)Cg!Bu&ob`IBum%39M}X|KCI5bByWMU4rC=dA1Pr!45~4 KZAG62B>xLWEf`+_ literal 8724 zcmeHNYgAKL8a)YbilW7eQzTfmQbYk;Dj*PW#8Rw)RS`shJrEL0O1XR5azppOqrQZTV|$fboh~EoqO-OA@_WH ze|zu4$sG>13sm1#MF=gh`_yJ9LUJ##m$D)pvCsVO8GNXOe7ZLjA$4!;h0C$Z_Cx5s zZ|rO~?~c6M#jE${KCfi;`W|AfRHUi$j=fN<2oh&JtSMU}uF(3xZ*_7n{woA~~cUY!~{}i4fEBb14Vl9rPfYLbS3yi z@|)nzW)b{VXpwb>9GpQ33t@npJRv`SV*Zi5uEk(_!xE{By1Vn-+}$s5IGhht>-=eQ z1tuF)?aLSpgZeZ}eI+G^&csK;HSwFSruHchScZm$Io-+2wNzw2b-lGmAn6y#vT+z9?!=ZVCSiwYV+_rgxY*I>xny#T3+1$}=%q%$@Rw;svvpQ+6JcE&2)9yA zO42M4BAO)-N%oxNEI)r*760mTooJhjN*|}zNx#LRKZMd1Q9*VE^YC&*9#=cS?5yS` zheV*KC%Zhnwy9QrVmx^uxp{b8EGjDM6Qp=H5){zNkt+%)r(G(oz?gy|{;ai_ z&T6K3cx{3yv8IMb+N`&>XWN0YXj90tfbp@2sC&Ph@qRr=zWy^pc2j^ey_eM*skM$X z#QF{(L6~EuM~Ni1;OpCgQ&tuO~&?q zSF(nd!B{F6F~?@}`aj6Iw%J*~e0c*Qp5-$e)7W~b44l+D{-s)V0Z%+8%cKpCb_JSI zdi^zhfPs5V{wZu>1^)(HzTKZD2I`yz&qhK9NJ9-A-A0hzBl@pE8Xn6%-%(8_cd|ev zcsX0N&x>*}D|59YZ8<|I#`VYQ2U_AVNr~Eik-guqHM~5`-Fd1dz6bPN=#o4beV4Z< z*Qx6qK&Uj)maCiRcjd9Meb)vUk0^D&xkd>S6o|>1QJ5C4Cc!tgt=|SyXpu9+NmipS z`TVCBi!}4YH!MZFg^MFl+vfEt?8n7j;i5Zvz(k9xv=3v)mL`!&q(Z<*tnN7($Hemj zRP`fPv$JEmi+Owt$ex~zD_MH>oK9(}E-O*+?8)^7h?Zi@1r*IeAkRYXl-8cbri0P$ z+w`uB$JB|9b*cPXUP#fnIL>^iL#{K?Jw`SzD=Tk46uzl-iIlPCfjyc?3*{KsZIo?X z(^W2J=o@-6Zdxj$317PF1z~-96Jb@YVWo8Ev|k_x)k5ubiz6P2ySr=iD`l?4328rg z4&q4Uq#~~Mr?2go#H3_)D~L?a4r;MR&wLaRQRaoaK>ny4#YEijhGqxO-pSjzCaZr4{#+;Clkb?8(Cb7jVnCVhU zy2&?vV9)tJ0P7hu-a~j4WFKREs5OF+D&)^x*l3tinTpM53|@}iIF(>{68cX(dV_0z zuB`H#=Ov&onbmj7%k`a|odp8}w=qrGfjZCB9oUtZpU*Dxt*@<05VIz}1=G1um}P;d z)??l`PvlR-6_fyr@QWuKaiIr20ASo#cGgj9{cTQT`7CX&DI!+m*&nNI`Q35va{e&22GURzsS^RKQRa5BIIv`2QyASA4=$I0 zr?xLm+PXKtsWp)9I_+{J?@`+)`^k3%jJ4w07GnA4oOQ!*j_|qW0RJTli_dJ4V=|t+ zikl|I058vlO^YCR-c~^|oAQj;{EVA@G@3pk%Wqi)*7rHY4UjEe4{Ysf40E5>R_0S4 zrlzI_3^C;#p)EU@u@OH1%OPc<{{rOY7?tqS)Zd1GIP1Rv`v4aB2PJwv1SkAWYxFf_Wh6l-WVN-HYX z`TF__*oKuiEme_Ch?4fdw6Vt`R3N&mU?wXG6|#B^Xr)kI^Rd-Wmg_u<|A$BiLvhat zw?cAjNSXGyF0~ayBdU@gy|Q&mm~dq~Q=9 z%OE3n%*xYjE(L-L?RqcL6zp1YAUpc5LBFJ8$P`hkg<_rV@}p0~cscam6JwE%>B}!R zAKF7vYr~nD2zsH$Uzyl}OS&urKNKnb59T$qiXgv9v~!ytmEZKPikIP(UtR0?N8#|t zq#%5x%j5>sJ@DaueFAobtU`20m4ia4RCi42ev&K_iFTfXjjHtz?9C2nd8TY{;fpi) zJl+M*xN&-nOd^DxBx+CZ}Gnf*|i;j_I0yyFsx0{%62ManA z0bEmu^hGk%8qAM6lomI0p-ZKkEC}<*Q!ckLIJ#mrR^#|LxYYmOHR)^f{|RkzIDwMp Wxu+L4{{>1C*==>O$@{(6k-q{?F4JTH diff --git a/tests/baseline/test_log_mpl39.png b/tests/baseline/test_log_mpl39.png new file mode 100644 index 0000000000000000000000000000000000000000..4a267e5726c6ca952cb7dd1a7153dfa3260418b4 GIT binary patch literal 8724 zcmeHNYgAKL8a)YbilW7eQzTfmQbYk;Dj*PW#8Rw)RS`shJrEL0O1XR5azppOqrQZTV|$fboh~EoqO-OA@_WH ze|zu4$sG>13sm1#MF=gh`_yJ9LUJ##m$D)pvCsVO8GNXOe7ZLjA$4!;h0C$Z_Cx5s zZ|rO~?~c6M#jE${KCfi;`W|AfRHUi$j=fN<2oh&JtSMU}uF(3xZ*_7n{woA~~cUY!~{}i4fEBb14Vl9rPfYLbS3yi z@|)nzW)b{VXpwb>9GpQ33t@npJRv`SV*Zi5uEk(_!xE{By1Vn-+}$s5IGhht>-=eQ z1tuF)?aLSpgZeZ}eI+G^&csK;HSwFSruHchScZm$Io-+2wNzw2b-lGmAn6y#vT+z9?!=ZVCSiwYV+_rgxY*I>xny#T3+1$}=%q%$@Rw;svvpQ+6JcE&2)9yA zO42M4BAO)-N%oxNEI)r*760mTooJhjN*|}zNx#LRKZMd1Q9*VE^YC&*9#=cS?5yS` zheV*KC%Zhnwy9QrVmx^uxp{b8EGjDM6Qp=H5){zNkt+%)r(G(oz?gy|{;ai_ z&T6K3cx{3yv8IMb+N`&>XWN0YXj90tfbp@2sC&Ph@qRr=zWy^pc2j^ey_eM*skM$X z#QF{(L6~EuM~Ni1;OpCgQ&tuO~&?q zSF(nd!B{F6F~?@}`aj6Iw%J*~e0c*Qp5-$e)7W~b44l+D{-s)V0Z%+8%cKpCb_JSI zdi^zhfPs5V{wZu>1^)(HzTKZD2I`yz&qhK9NJ9-A-A0hzBl@pE8Xn6%-%(8_cd|ev zcsX0N&x>*}D|59YZ8<|I#`VYQ2U_AVNr~Eik-guqHM~5`-Fd1dz6bPN=#o4beV4Z< z*Qx6qK&Uj)maCiRcjd9Meb)vUk0^D&xkd>S6o|>1QJ5C4Cc!tgt=|SyXpu9+NmipS z`TVCBi!}4YH!MZFg^MFl+vfEt?8n7j;i5Zvz(k9xv=3v)mL`!&q(Z<*tnN7($Hemj zRP`fPv$JEmi+Owt$ex~zD_MH>oK9(}E-O*+?8)^7h?Zi@1r*IeAkRYXl-8cbri0P$ z+w`uB$JB|9b*cPXUP#fnIL>^iL#{K?Jw`SzD=Tk46uzl-iIlPCfjyc?3*{KsZIo?X z(^W2J=o@-6Zdxj$317PF1z~-96Jb@YVWo8Ev|k_x)k5ubiz6P2ySr=iD`l?4328rg z4&q4Uq#~~Mr?2go#H3_)D~L?a4r;MR&wLaRQRaoaK>ny4#YEijhGqxO-pSjzCaZr4{#+;Clkb?8(Cb7jVnCVhU zy2&?vV9)tJ0P7hu-a~j4WFKREs5OF+D&)^x*l3tinTpM53|@}iIF(>{68cX(dV_0z zuB`H#=Ov&onbmj7%k`a|odp8}w=qrGfjZCB9oUtZpU*Dxt*@<05VIz}1=G1um}P;d z)??l`PvlR-6_fyr@QWuKaiIr20ASo#cGgj9{cTQT`7CX&DI!+m*&nNI`Q35va{e&22GURzsS^RKQRa5BIIv`2QyASA4=$I0 zr?xLm+PXKtsWp)9I_+{J?@`+)`^k3%jJ4w07GnA4oOQ!*j_|qW0RJTli_dJ4V=|t+ zikl|I058vlO^YCR-c~^|oAQj;{EVA@G@3pk%Wqi)*7rHY4UjEe4{Ysf40E5>R_0S4 zrlzI_3^C;#p)EU@u@OH1%OPc<{{rOY7?tqS)Zd1GIP1Rv`v4aB2PJwv1SkAWYxFf_Wh6l-WVN-HYX z`TF__*oKuiEme_Ch?4fdw6Vt`R3N&mU?wXG6|#B^Xr)kI^Rd-Wmg_u<|A$BiLvhat zw?cAjNSXGyF0~ayBdU@gy|Q&mm~dq~Q=9 z%OE3n%*xYjE(L-L?RqcL6zp1YAUpc5LBFJ8$P`hkg<_rV@}p0~cscam6JwE%>B}!R zAKF7vYr~nD2zsH$Uzyl}OS&urKNKnb59T$qiXgv9v~!ytmEZKPikIP(UtR0?N8#|t zq#%5x%j5>sJ@DaueFAobtU`20m4ia4RCi42ev&K_iFTfXjjHtz?9C2nd8TY{;fpi) zJl+M*xN&-nOd^DxBx+CZ}Gnf*|i;j_I0yyFsx0{%62ManA z0bEmu^hGk%8qAM6lomI0p-ZKkEC}<*Q!ckLIJ#mrR^#|LxYYmOHR)^f{|RkzIDwMp Wxu+L4{{>1C*==>O$@{(6k-q{?F4JTH literal 0 HcmV?d00001 diff --git a/tests/test_basic.py b/tests/test_basic.py index 9960b21f..25df5ac5 100644 --- a/tests/test_basic.py +++ b/tests/test_basic.py @@ -4,6 +4,7 @@ import re import hist +import matplotlib as mpl import matplotlib.pyplot as plt import numpy as np import pytest @@ -59,6 +60,26 @@ def test_simple2d(): return fig +@pytest.mark.skipif( + (int(mpl.__version__.split(".")[0]), int(mpl.__version__.split(".")[1])) >= (3, 10), + reason="Change in mpl behaviour since 3.10", +) +@pytest.mark.mpl_image_compare(style="default", remove_text=True) +def test_log_mpl39(): + fig, axs = plt.subplots(2, 2, figsize=(10, 10)) + for ax in axs[0]: + hep.histplot([1, 2, 3, 2], range(5), ax=ax) + ax.semilogy() + for ax in axs[1]: + hep.histplot([1, 2, 3, 2], range(5), ax=ax, edges=False) + ax.semilogy() + return fig + + +@pytest.mark.skipif( + (int(mpl.__version__.split(".")[0]), int(mpl.__version__.split(".")[1])) < (3, 10), + reason="Change in mpl behaviour since 3.10", +) @pytest.mark.mpl_image_compare(style="default", remove_text=True) def test_log(): fig, axs = plt.subplots(2, 2, figsize=(10, 10))