From a6957ae1b0e5de534a5d055bad8c7d4630e8d505 Mon Sep 17 00:00:00 2001 From: AngeloDanducci Date: Thu, 26 Oct 2023 14:23:41 -0400 Subject: [PATCH 1/7] only publish images if image tests run --- .azure/test-linux.yml | 12 ++++++++---- .../references/5_qubit_gate_map copy.png | Bin 0 -> 5329 bytes .../mpl/graph/references/5_qubit_gate_map.png | Bin 5329 -> 1183 bytes 3 files changed, 8 insertions(+), 4 deletions(-) create mode 100644 test/visual/mpl/graph/references/5_qubit_gate_map copy.png diff --git a/.azure/test-linux.yml b/.azure/test-linux.yml index c97d4f693975..fec95d334fd9 100644 --- a/.azure/test-linux.yml +++ b/.azure/test-linux.yml @@ -29,6 +29,7 @@ jobs: QISKIT_SUPPRESS_PACKAGING_WARNINGS: Y PIP_CACHE_DIR: $(Pipeline.Workspace)/.pip QISKIT_TEST_CAPTURE_STREAMS: 1 + HAVE_VISUAL_TESTS_RUN: false steps: - task: UsePythonVersion@0 @@ -204,6 +205,9 @@ jobs: # a new internal interface that will be the default in jupyter-core 6.x. # This variable should become redundant on release of jupyter-core 6. JUPYTER_PLATFORM_DIRS: 1 + + - bash: | + echo "##vso[task.setvariable variable=HAVE_VISUAL_TESTS_RUN;]true" - task: ArchiveFiles@2 displayName: Archive visual test failure diffs @@ -213,7 +217,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/visual_test_failures.tar.gz' verbose: true - condition: failed() + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, true)) - task: ArchiveFiles@2 displayName: Archive circuit results @@ -222,7 +226,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/circuit_results.tar.gz' verbose: true - condition: failed() + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, true)) - task: ArchiveFiles@2 displayName: Archive graph results @@ -231,7 +235,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/graph_results.tar.gz' verbose: true - condition: failed() + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, true)) - task: PublishBuildArtifacts@1 displayName: 'Publish image test failure diffs' @@ -240,4 +244,4 @@ jobs: artifactName: 'image_test_failure_img_diffs' Parallel: true ParallelCount: 8 - condition: failed() + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, true)) diff --git a/test/visual/mpl/graph/references/5_qubit_gate_map copy.png b/test/visual/mpl/graph/references/5_qubit_gate_map copy.png new file mode 100644 index 0000000000000000000000000000000000000000..1bc197fa0556f734c3435295255cdfe6f9bc120d GIT binary patch literal 5329 zcmZ{I2T&93w?2rVV(2ZjNC`!x1nDh&RDnD7-n09@vwO~iF*m)%K+i)@MMcG6XrOx= zxUZg{moEU)HtK$#fk&H46A14)dtm>LBBAESu-xcEl zyK=oHh^Wnd^_5pJx2w(ewc^ZBb^yg@2HTJK8xvBDu}O=s27a|nsyhYD+Z5PXs#Km&) z-vgIQ(_hEg%;UO+F)ZiE;SSg7QzygNwQR+D#cp>qS^3*d|VKhh4}{T-R%27 zsY(^ph6bCs#AMd~XqDf&1d0hu3{!|~DJD4Jy&gfgKVD(Z=(O)_@VyVTYiL6e7A-L<78>4AEdKxw(+pwBsH+FnhRL+ z)ptdNB!l&~)#W5kMw%3gX(qux+!n>dS%$^^-{yZdMZ~D3*?i_y<{f= zR?lR&ZDX}t_N@hE9IP$Yr}` zyY740^vlbqPW=2)cSD5>b*P>&Yi!!{)ADexn*|fpu?hBddibY*B2PlZoY`Z?KKMal zr@!1YY+2e?i3a72uXk;s)=2?jlB{L=mo+#s9MA`lWYT?B39KH{ z;|dmlcRnQna8%ng0Ec&mOR^?hr~!aqV%h+pp!OZC-fY0@f5}3qMe{!Q4CNH@1)>4p zZL4itD5Fd#8i4aCr~~k{3jkbu8t|mOTn~gzht>lFZ44Kdi3XD0!incriRc|$D5vD* zOu+7H@&s_Bb%>b-?8+>cfqVoRPWJ)(;9StTB76)yfaKjI|MQSse}{l10}`0dmC?UF zfU+4=c^&}!m+amnZ^D$oElLv$0^L+(G;c^rx+&4wU&B`?^XjP>eu$#N?iqN0GI`)K z(?nk{&(EU3Jggcf;s5yLHhd6Wd@6_U*66i_=NSFU5}54lFqMb3tP7YRnsRvR28RFf z$Vnz#%uT1@zRxu*;GB_PUKJN)D$3)o3m0ieCo=N%Fs#1Qu?d@&I_Ta!RB7TeTXQL5 zVO*cnc8T0E6h(yKft7L5f4O6Q#R`~MP)nx#89JygnLpY)v*sw_|LY>+r#|`{877hdWU)3k~^WR zUFG66lTqW-B{JK;$~g1byF9p^&c@I}(7&M|EYbU>tl_>$ z@Kij3#D{Z&hr_i2gEMrp(=76frA%OXY{=1%s@>4_x|=bQ~8EQYOR*>vyhBB9zJi z-&Rlbh!L{=`0(s+DDReath z%g&x(X0Ci_Y4uMJPbL=%^#nol=8`JK9W^_e^{Tih4C>tcVi5VH#Kq__PUFPEn9XT! zuFm0%42l(g2LjpqGcYqAL^Q_*Wl1?VD|TP4q<+pxd^7%O3^qU4g7Ak%+1j`@#_GU1 zS;-E*KI0#+xIWC60vZ@?@_=PDg9b(_kyOPyI_QDA)4 zgjn~1`~zB;Nw1`GNp8C|-U{>clO%!vo~`rq4?!~rbD)!J4bG2nzG}u^8}BEUqxiN_ z1i!0KyzQ|{_pUg=MY_hSa%Fz$?C?|FatN)P@{a0?D7@Z`Uv6E0YqWIfWTn4!moe7; zO}I1(wzN32USP_wE5~@CT;+3BVT7u(;RT_H!4Mv`ExQ--aPo$KLr>2T;Sl#zdZ@QN9daZm|Dh^%^I`)Ha36D zeOZ1{Ue@i(k#`5a6E=JLw0&GCIq-clIxwkBC))AonHxOF&Y3j2Z}}d?_A)}cs3QS3 zAM{r_0ZYTq@xJ2OXTOpyorq;q#me7XMc(vw*?kQ!(XQW?rnfrgnd08Kr$eh(iYm?Q zH(-e7p3-rOGNn4*gH}CIAT)jx=d$y5#3K};{fsH#am+g6c#}KwP^cwAX|n_Sucm|u zK9tp^P7@W+n+02~IDNrUvhJM_q}SAbr}}E)aeJhqH8r0~hm6_pGNH3{lWP3qxH288 z&vhRwzO_rVWUO@Hp5=AD3n@LG-tDq^7jl`2r6Bg`Z@&~(UsKxv4lyAx<(`}0e4*-o z+=@HJ(9tpb-o{QAwW771OWR8o>SopFk%Ve-C%sdsFI;Ys$K^-Y8ZUL^E{_?1h*QS+ z!5GUxYvj1A^!0;oobuZJmb7XrU}2l!k1X}0~^jk4X3t?I{ZSuc&@Tb6wC%)rqWr?@e zd114}yR0*~KofE5w?0M}P>74tj8DMu)$9=GTGr=3T04C5(YU&k<5^f?;js3BXY){Z zTQcxO0;1CLu1q}feVB;LxXy|76ZeuqP@=+EgJ`MJ389A)jYW;FQOPuS;LcV&q;Ic9 zG@YoN{$cvKX8qRs_Ir?1N%>R$JoG$Ol+~q8(`#ha{I^xzVvlMccuM&@A;;k-TvM(d zDGU0w6P+$mev^dnQ^nWLymN;New~s9uQ^egm@2eKd<~sFoxk-t&vLER{=SjVRP{E0 zx9ow~fGM7m&_~FD)^>~CnLU)^J2lCThiDj5XxAsT*6fk(DQQ778fo}mFNoF9n8(g2 z!^NC=9K(Iej*=$wD6F>9bJwUd%kuWO-H!P~a;@~INfVbQ4>)I?n8rc=?u6+Nwt(UFXqpX;8bxC^2@xO*5rHvd43{+KFl z(`fu}_%6ri#z(_e#ioTGI=Y^|v_22k)vxd!d@HHVJz5U5o*sN3%8+L*;|%}4yQs>f zR9z^qw3fI?GD6F=vPmJ%P8d}LDF@{#sH*Fwaz5Ad;eDFNN?y-ptl_mU0MMUkzT<*> zAPBE#1ab+VjMoXw?&}0k9j2^lIJF}8X0B^AKF^wOnKRxt+yR%pwqR^m30K**pyDoE zDKDEgLQbZ7c;k<_1>W}o`NW#M>Q48$1scihAuEwgj$jcJ%~=HUrP$jF_JKLZssv812MC^|L}+h+x89xtBU zuj^~8KVPV4{n_JNaLq7*)4T4A!RxX7c>;Y0_2J+4{$J@hj(jE1xlpq@rhAe(9Ip3e zTGno!&1;ZH9r1Fz<8Xyhlq%`!8_5!(1$i|vE7lWc9Qo(Qr%A)f!BU@yMNW8B9X-|_ zAs7@(nQgsxN!@5Q)jah0gwr{4OT#qGXXSZfq&11Jd0-$8Q0EVw; z0EmMBhu2sDn50)sODX#mFiHM3c33u*At3k+!?In^1-0k0*)8Y7RjZrX0zkxk{v>YW ze?)-PlEUcN95F1|iK2g7FWLinD3#uaX=Q=8l;zkJ!L&eN(}Z!lqIh7=J#l|nCosVS zJupaL1n{A-vBp{$BS~{im$9v-VzRTSZy@?y{r#au}syD4VF3{sXM3BO#mg=Ck*BfaSS#&(JVTwH@xwZN!0%^5n z3wnuIgo^$J2ChG~K*p>R!42u@M#eA=&RD4{m86S3?y_TP<_O~ebBZ^AKds{BH9;renWWG~wAM#v|8fx?b6A5U>javCHo zEGO4gN{r`qZEk_Wu)m1)GYbx6fM_C#JLCIY)5R@o)8N8r>fqP+c|Otg*QXNbB)anH zxz5w+eLmW^)rh`-#*@0)V>a+8l1V4_Y<2$KCS5OXL4#Y{ zq=PIuhD0|SvhFqbBoK(TF{c+tDWF%yOu@-ObN&(HUp_kyi!CzlTY6%X`JO0o;Q!@* zCI{8=G}ng^J1+3Rb*Xz1LQ6#OYP8p((^Hl1ZX3R&AcV*Ulbo_*-*k+4)V@auttl@E zYhe$9c_g+v1FiYa3`4{mRRXqTul0#{pl^0imP!avFI~Seb0d6+OL-2pYr;1sMHj9G z-P0b`leSHbEg^*d!TTTQ9_0M^&{|yPxXOiT6yQ>G96GPa=5q3B;#)JU>F9zSAAo`Q zZ2mnC1EO2C4ICBHLPWKAeU%>_e`3l|DsJP`B<}dH+qdi6fHvD}MZ3%MK(rQzis1)f zrn~7|cT{A+3xiX+Jjy_44i?pDtX&Zbj5Q4LU+?1?(T4_WMYSP)?R#_bXP3<3Wf`|*0_zP%N|vqS@b!_>?^L@>_-XIul5**E5o)nE9@s>;WC!ycrdgXNRe z@mVYFTTIw%H4|@rBKGDYkO}*o@ytt@35yMJuRSmO-2Eq2faKtpSx zSm{9)U9N*(YZ9Hmn4v5K{!e@PpT3`XEDfQT5+X9A?Y3_RXuHPz7qwG50ssI2 literal 0 HcmV?d00001 diff --git a/test/visual/mpl/graph/references/5_qubit_gate_map.png b/test/visual/mpl/graph/references/5_qubit_gate_map.png index 1bc197fa0556f734c3435295255cdfe6f9bc120d..d04aca637ddf420b73d49cee4c7f419e7ae24f15 100644 GIT binary patch delta 1180 zcmV;N1Y`TrDW3@;iBL{Q4GJ0x0000DNk~Le0000i0000i2m$~A0Fma+R*@kTe*{!X zL_t(oh3%M2Y#T)w$A7bqlQ?AE=0R~%sU?k@D2+r?DRE5bfrp?f#8pL$RP}!2-A& zd94F?BhD&T*9k2$^oX$;i4mK^(Q5jyP-nh1rVz1JKK+(hl5~Ls`n>qO6vlynim!i= ztXYQR1;BH zr5J+&xhHyBsdfO|`OS!U=XZ1+MeyoN@|iS^yMCha(jW%>aw|aRDS+e#6CvZPXq?9p zfV0h}!_gTClmL-PGBPP~#*AKc0X&Y#lT|PbP^+Rkq{|thvi(m)Gad7VRYIGVl0SRR;=>)^3b)mwZeMp&X zR%$EzP-&NU0MMBU8@)acVmT(MjS@QtI}oO}SV@>eMDyPnw2{g_l$+Fld#-&wCGW7U z<$SNogIM1zHu6bsf6L9XuhU`dX(q!&dpO>$u4kj``6KcU^||)QO6gV$AROfuz5k8? zG`SG6IBsqLSV>4SfkBn|6Xih^skRx7&jM6c(0=|*EgH%42ho4@8sn`eUIN7ShJytJL*lWTde{8QW>DnOe^WZBm{jPNxKB1-ifcLn0xL|io!*&JPXJ(=7lbrUB)0h zh-?b(2F`bx@X1>hZumG_eK;I8g&LLXeWt(5rjw{u^j|pyjts#^mrc)nSpEhkl6*D5 zYy_?3ztG)M6xobR%}v?%pzAR1>Nz=H{N z&~z}n8UP^>g5GxC>9($jdxOa6$r^!a#)Hg+cTSYsvX^oCI31_(v=lq*9u#6*JJ&^< u0FH9%k5H%XGrQ~);&Ao9xV?J@k<-6yLVKK9rMKPy0000D7-n09@vwO~iF*m)%K+i)@MMcG6XrOx= zxUZg{moEU)HtK$#fk&H46A14)dtm>LBBAESu-xcEl zyK=oHh^Wnd^_5pJx2w(ewc^ZBb^yg@2HTJK8xvBDu}O=s27a|nsyhYD+Z5PXs#Km&) z-vgIQ(_hEg%;UO+F)ZiE;SSg7QzygNwQR+D#cp>qS^3*d|VKhh4}{T-R%27 zsY(^ph6bCs#AMd~XqDf&1d0hu3{!|~DJD4Jy&gfgKVD(Z=(O)_@VyVTYiL6e7A-L<78>4AEdKxw(+pwBsH+FnhRL+ z)ptdNB!l&~)#W5kMw%3gX(qux+!n>dS%$^^-{yZdMZ~D3*?i_y<{f= zR?lR&ZDX}t_N@hE9IP$Yr}` zyY740^vlbqPW=2)cSD5>b*P>&Yi!!{)ADexn*|fpu?hBddibY*B2PlZoY`Z?KKMal zr@!1YY+2e?i3a72uXk;s)=2?jlB{L=mo+#s9MA`lWYT?B39KH{ z;|dmlcRnQna8%ng0Ec&mOR^?hr~!aqV%h+pp!OZC-fY0@f5}3qMe{!Q4CNH@1)>4p zZL4itD5Fd#8i4aCr~~k{3jkbu8t|mOTn~gzht>lFZ44Kdi3XD0!incriRc|$D5vD* zOu+7H@&s_Bb%>b-?8+>cfqVoRPWJ)(;9StTB76)yfaKjI|MQSse}{l10}`0dmC?UF zfU+4=c^&}!m+amnZ^D$oElLv$0^L+(G;c^rx+&4wU&B`?^XjP>eu$#N?iqN0GI`)K z(?nk{&(EU3Jggcf;s5yLHhd6Wd@6_U*66i_=NSFU5}54lFqMb3tP7YRnsRvR28RFf z$Vnz#%uT1@zRxu*;GB_PUKJN)D$3)o3m0ieCo=N%Fs#1Qu?d@&I_Ta!RB7TeTXQL5 zVO*cnc8T0E6h(yKft7L5f4O6Q#R`~MP)nx#89JygnLpY)v*sw_|LY>+r#|`{877hdWU)3k~^WR zUFG66lTqW-B{JK;$~g1byF9p^&c@I}(7&M|EYbU>tl_>$ z@Kij3#D{Z&hr_i2gEMrp(=76frA%OXY{=1%s@>4_x|=bQ~8EQYOR*>vyhBB9zJi z-&Rlbh!L{=`0(s+DDReath z%g&x(X0Ci_Y4uMJPbL=%^#nol=8`JK9W^_e^{Tih4C>tcVi5VH#Kq__PUFPEn9XT! zuFm0%42l(g2LjpqGcYqAL^Q_*Wl1?VD|TP4q<+pxd^7%O3^qU4g7Ak%+1j`@#_GU1 zS;-E*KI0#+xIWC60vZ@?@_=PDg9b(_kyOPyI_QDA)4 zgjn~1`~zB;Nw1`GNp8C|-U{>clO%!vo~`rq4?!~rbD)!J4bG2nzG}u^8}BEUqxiN_ z1i!0KyzQ|{_pUg=MY_hSa%Fz$?C?|FatN)P@{a0?D7@Z`Uv6E0YqWIfWTn4!moe7; zO}I1(wzN32USP_wE5~@CT;+3BVT7u(;RT_H!4Mv`ExQ--aPo$KLr>2T;Sl#zdZ@QN9daZm|Dh^%^I`)Ha36D zeOZ1{Ue@i(k#`5a6E=JLw0&GCIq-clIxwkBC))AonHxOF&Y3j2Z}}d?_A)}cs3QS3 zAM{r_0ZYTq@xJ2OXTOpyorq;q#me7XMc(vw*?kQ!(XQW?rnfrgnd08Kr$eh(iYm?Q zH(-e7p3-rOGNn4*gH}CIAT)jx=d$y5#3K};{fsH#am+g6c#}KwP^cwAX|n_Sucm|u zK9tp^P7@W+n+02~IDNrUvhJM_q}SAbr}}E)aeJhqH8r0~hm6_pGNH3{lWP3qxH288 z&vhRwzO_rVWUO@Hp5=AD3n@LG-tDq^7jl`2r6Bg`Z@&~(UsKxv4lyAx<(`}0e4*-o z+=@HJ(9tpb-o{QAwW771OWR8o>SopFk%Ve-C%sdsFI;Ys$K^-Y8ZUL^E{_?1h*QS+ z!5GUxYvj1A^!0;oobuZJmb7XrU}2l!k1X}0~^jk4X3t?I{ZSuc&@Tb6wC%)rqWr?@e zd114}yR0*~KofE5w?0M}P>74tj8DMu)$9=GTGr=3T04C5(YU&k<5^f?;js3BXY){Z zTQcxO0;1CLu1q}feVB;LxXy|76ZeuqP@=+EgJ`MJ389A)jYW;FQOPuS;LcV&q;Ic9 zG@YoN{$cvKX8qRs_Ir?1N%>R$JoG$Ol+~q8(`#ha{I^xzVvlMccuM&@A;;k-TvM(d zDGU0w6P+$mev^dnQ^nWLymN;New~s9uQ^egm@2eKd<~sFoxk-t&vLER{=SjVRP{E0 zx9ow~fGM7m&_~FD)^>~CnLU)^J2lCThiDj5XxAsT*6fk(DQQ778fo}mFNoF9n8(g2 z!^NC=9K(Iej*=$wD6F>9bJwUd%kuWO-H!P~a;@~INfVbQ4>)I?n8rc=?u6+Nwt(UFXqpX;8bxC^2@xO*5rHvd43{+KFl z(`fu}_%6ri#z(_e#ioTGI=Y^|v_22k)vxd!d@HHVJz5U5o*sN3%8+L*;|%}4yQs>f zR9z^qw3fI?GD6F=vPmJ%P8d}LDF@{#sH*Fwaz5Ad;eDFNN?y-ptl_mU0MMUkzT<*> zAPBE#1ab+VjMoXw?&}0k9j2^lIJF}8X0B^AKF^wOnKRxt+yR%pwqR^m30K**pyDoE zDKDEgLQbZ7c;k<_1>W}o`NW#M>Q48$1scihAuEwgj$jcJ%~=HUrP$jF_JKLZssv812MC^|L}+h+x89xtBU zuj^~8KVPV4{n_JNaLq7*)4T4A!RxX7c>;Y0_2J+4{$J@hj(jE1xlpq@rhAe(9Ip3e zTGno!&1;ZH9r1Fz<8Xyhlq%`!8_5!(1$i|vE7lWc9Qo(Qr%A)f!BU@yMNW8B9X-|_ zAs7@(nQgsxN!@5Q)jah0gwr{4OT#qGXXSZfq&11Jd0-$8Q0EVw; z0EmMBhu2sDn50)sODX#mFiHM3c33u*At3k+!?In^1-0k0*)8Y7RjZrX0zkxk{v>YW ze?)-PlEUcN95F1|iK2g7FWLinD3#uaX=Q=8l;zkJ!L&eN(}Z!lqIh7=J#l|nCosVS zJupaL1n{A-vBp{$BS~{im$9v-VzRTSZy@?y{r#au}syD4VF3{sXM3BO#mg=Ck*BfaSS#&(JVTwH@xwZN!0%^5n z3wnuIgo^$J2ChG~K*p>R!42u@M#eA=&RD4{m86S3?y_TP<_O~ebBZ^AKds{BH9;renWWG~wAM#v|8fx?b6A5U>javCHo zEGO4gN{r`qZEk_Wu)m1)GYbx6fM_C#JLCIY)5R@o)8N8r>fqP+c|Otg*QXNbB)anH zxz5w+eLmW^)rh`-#*@0)V>a+8l1V4_Y<2$KCS5OXL4#Y{ zq=PIuhD0|SvhFqbBoK(TF{c+tDWF%yOu@-ObN&(HUp_kyi!CzlTY6%X`JO0o;Q!@* zCI{8=G}ng^J1+3Rb*Xz1LQ6#OYP8p((^Hl1ZX3R&AcV*Ulbo_*-*k+4)V@auttl@E zYhe$9c_g+v1FiYa3`4{mRRXqTul0#{pl^0imP!avFI~Seb0d6+OL-2pYr;1sMHj9G z-P0b`leSHbEg^*d!TTTQ9_0M^&{|yPxXOiT6yQ>G96GPa=5q3B;#)JU>F9zSAAo`Q zZ2mnC1EO2C4ICBHLPWKAeU%>_e`3l|DsJP`B<}dH+qdi6fHvD}MZ3%MK(rQzis1)f zrn~7|cT{A+3xiX+Jjy_44i?pDtX&Zbj5Q4LU+?1?(T4_WMYSP)?R#_bXP3<3Wf`|*0_zP%N|vqS@b!_>?^L@>_-XIul5**E5o)nE9@s>;WC!ycrdgXNRe z@mVYFTTIw%H4|@rBKGDYkO}*o@ytt@35yMJuRSmO-2Eq2faKtpSx zSm{9)U9N*(YZ9Hmn4v5K{!e@PpT3`XEDfQT5+X9A?Y3_RXuHPz7qwG50ssI2 From 3c08096bad2370e79d6a4a558693154204be14b5 Mon Sep 17 00:00:00 2001 From: AngeloDanducci Date: Thu, 26 Oct 2023 15:00:53 -0400 Subject: [PATCH 2/7] fix indent in bash command --- .azure/test-linux.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.azure/test-linux.yml b/.azure/test-linux.yml index fec95d334fd9..68aa57ac2bce 100644 --- a/.azure/test-linux.yml +++ b/.azure/test-linux.yml @@ -207,7 +207,7 @@ jobs: JUPYTER_PLATFORM_DIRS: 1 - bash: | - echo "##vso[task.setvariable variable=HAVE_VISUAL_TESTS_RUN;]true" + echo "##vso[task.setvariable variable=HAVE_VISUAL_TESTS_RUN;]true" - task: ArchiveFiles@2 displayName: Archive visual test failure diffs From 6a914206e1ca0f8186c20331692fda99d1e3e03b Mon Sep 17 00:00:00 2001 From: AngeloDanducci Date: Thu, 26 Oct 2023 16:24:34 -0400 Subject: [PATCH 3/7] fix condition syntax --- .azure/test-linux.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.azure/test-linux.yml b/.azure/test-linux.yml index 68aa57ac2bce..3d39d2da5930 100644 --- a/.azure/test-linux.yml +++ b/.azure/test-linux.yml @@ -217,7 +217,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/visual_test_failures.tar.gz' verbose: true - condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, true)) + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) - task: ArchiveFiles@2 displayName: Archive circuit results @@ -226,7 +226,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/circuit_results.tar.gz' verbose: true - condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, true)) + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) - task: ArchiveFiles@2 displayName: Archive graph results @@ -235,7 +235,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/graph_results.tar.gz' verbose: true - condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, true)) + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) - task: PublishBuildArtifacts@1 displayName: 'Publish image test failure diffs' @@ -244,4 +244,4 @@ jobs: artifactName: 'image_test_failure_img_diffs' Parallel: true ParallelCount: 8 - condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, true)) + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) From df713f0f7128f04bd0ac2ca8a2394604a3973d7d Mon Sep 17 00:00:00 2001 From: AngeloDanducci Date: Thu, 26 Oct 2023 18:00:54 -0400 Subject: [PATCH 4/7] set value in the image test --- .azure/test-linux.yml | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/.azure/test-linux.yml b/.azure/test-linux.yml index 3d39d2da5930..1934a6f2929a 100644 --- a/.azure/test-linux.yml +++ b/.azure/test-linux.yml @@ -198,17 +198,16 @@ jobs: env: SETUPTOOLS_ENABLE_FEATURES: "legacy-editable" - - bash: image_tests/bin/python -m unittest discover -v test/visual + - bash: | + echo "##vso[task.setvariable variable=HAVE_VISUAL_TESTS_RUN;]true" + image_tests/bin/python -m unittest discover -v test/visual displayName: 'Run image test' env: # Needed to suppress a warning in jupyter-core 5.x by eagerly migrating to # a new internal interface that will be the default in jupyter-core 6.x. # This variable should become redundant on release of jupyter-core 6. JUPYTER_PLATFORM_DIRS: 1 - - - bash: | - echo "##vso[task.setvariable variable=HAVE_VISUAL_TESTS_RUN;]true" - + - task: ArchiveFiles@2 displayName: Archive visual test failure diffs inputs: From 9cdae9f7fb3eb50e6a6215a18d1fdebfdecbe8f5 Mon Sep 17 00:00:00 2001 From: AngeloDanducci Date: Thu, 26 Oct 2023 22:29:21 -0400 Subject: [PATCH 5/7] test runner stopping correctly on fail --- test/python/transpiler/test_dense_layout.py | 2 +- .../references/5_qubit_gate_map copy.png | Bin 5329 -> 0 bytes .../mpl/graph/references/5_qubit_gate_map.png | Bin 1183 -> 5329 bytes 3 files changed, 1 insertion(+), 1 deletion(-) delete mode 100644 test/visual/mpl/graph/references/5_qubit_gate_map copy.png diff --git a/test/python/transpiler/test_dense_layout.py b/test/python/transpiler/test_dense_layout.py index 669fd485b358..57ea5bdef4b4 100644 --- a/test/python/transpiler/test_dense_layout.py +++ b/test/python/transpiler/test_dense_layout.py @@ -66,7 +66,7 @@ def test_finds_densest_component(self): pass_(circuit) layout = pass_.property_set["layout"] used_qubits = {layout[q] for q in circuit.qubits} - self.assertEqual(used_qubits, set(middle_qubits)) + self.assertEqual(0, set(middle_qubits)) def test_5q_circuit_20q_coupling(self): """Test finds dense 5q corner in 20q coupling map.""" diff --git a/test/visual/mpl/graph/references/5_qubit_gate_map copy.png b/test/visual/mpl/graph/references/5_qubit_gate_map copy.png deleted file mode 100644 index 1bc197fa0556f734c3435295255cdfe6f9bc120d..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 5329 zcmZ{I2T&93w?2rVV(2ZjNC`!x1nDh&RDnD7-n09@vwO~iF*m)%K+i)@MMcG6XrOx= zxUZg{moEU)HtK$#fk&H46A14)dtm>LBBAESu-xcEl zyK=oHh^Wnd^_5pJx2w(ewc^ZBb^yg@2HTJK8xvBDu}O=s27a|nsyhYD+Z5PXs#Km&) z-vgIQ(_hEg%;UO+F)ZiE;SSg7QzygNwQR+D#cp>qS^3*d|VKhh4}{T-R%27 zsY(^ph6bCs#AMd~XqDf&1d0hu3{!|~DJD4Jy&gfgKVD(Z=(O)_@VyVTYiL6e7A-L<78>4AEdKxw(+pwBsH+FnhRL+ z)ptdNB!l&~)#W5kMw%3gX(qux+!n>dS%$^^-{yZdMZ~D3*?i_y<{f= zR?lR&ZDX}t_N@hE9IP$Yr}` zyY740^vlbqPW=2)cSD5>b*P>&Yi!!{)ADexn*|fpu?hBddibY*B2PlZoY`Z?KKMal zr@!1YY+2e?i3a72uXk;s)=2?jlB{L=mo+#s9MA`lWYT?B39KH{ z;|dmlcRnQna8%ng0Ec&mOR^?hr~!aqV%h+pp!OZC-fY0@f5}3qMe{!Q4CNH@1)>4p zZL4itD5Fd#8i4aCr~~k{3jkbu8t|mOTn~gzht>lFZ44Kdi3XD0!incriRc|$D5vD* zOu+7H@&s_Bb%>b-?8+>cfqVoRPWJ)(;9StTB76)yfaKjI|MQSse}{l10}`0dmC?UF zfU+4=c^&}!m+amnZ^D$oElLv$0^L+(G;c^rx+&4wU&B`?^XjP>eu$#N?iqN0GI`)K z(?nk{&(EU3Jggcf;s5yLHhd6Wd@6_U*66i_=NSFU5}54lFqMb3tP7YRnsRvR28RFf z$Vnz#%uT1@zRxu*;GB_PUKJN)D$3)o3m0ieCo=N%Fs#1Qu?d@&I_Ta!RB7TeTXQL5 zVO*cnc8T0E6h(yKft7L5f4O6Q#R`~MP)nx#89JygnLpY)v*sw_|LY>+r#|`{877hdWU)3k~^WR zUFG66lTqW-B{JK;$~g1byF9p^&c@I}(7&M|EYbU>tl_>$ z@Kij3#D{Z&hr_i2gEMrp(=76frA%OXY{=1%s@>4_x|=bQ~8EQYOR*>vyhBB9zJi z-&Rlbh!L{=`0(s+DDReath z%g&x(X0Ci_Y4uMJPbL=%^#nol=8`JK9W^_e^{Tih4C>tcVi5VH#Kq__PUFPEn9XT! zuFm0%42l(g2LjpqGcYqAL^Q_*Wl1?VD|TP4q<+pxd^7%O3^qU4g7Ak%+1j`@#_GU1 zS;-E*KI0#+xIWC60vZ@?@_=PDg9b(_kyOPyI_QDA)4 zgjn~1`~zB;Nw1`GNp8C|-U{>clO%!vo~`rq4?!~rbD)!J4bG2nzG}u^8}BEUqxiN_ z1i!0KyzQ|{_pUg=MY_hSa%Fz$?C?|FatN)P@{a0?D7@Z`Uv6E0YqWIfWTn4!moe7; zO}I1(wzN32USP_wE5~@CT;+3BVT7u(;RT_H!4Mv`ExQ--aPo$KLr>2T;Sl#zdZ@QN9daZm|Dh^%^I`)Ha36D zeOZ1{Ue@i(k#`5a6E=JLw0&GCIq-clIxwkBC))AonHxOF&Y3j2Z}}d?_A)}cs3QS3 zAM{r_0ZYTq@xJ2OXTOpyorq;q#me7XMc(vw*?kQ!(XQW?rnfrgnd08Kr$eh(iYm?Q zH(-e7p3-rOGNn4*gH}CIAT)jx=d$y5#3K};{fsH#am+g6c#}KwP^cwAX|n_Sucm|u zK9tp^P7@W+n+02~IDNrUvhJM_q}SAbr}}E)aeJhqH8r0~hm6_pGNH3{lWP3qxH288 z&vhRwzO_rVWUO@Hp5=AD3n@LG-tDq^7jl`2r6Bg`Z@&~(UsKxv4lyAx<(`}0e4*-o z+=@HJ(9tpb-o{QAwW771OWR8o>SopFk%Ve-C%sdsFI;Ys$K^-Y8ZUL^E{_?1h*QS+ z!5GUxYvj1A^!0;oobuZJmb7XrU}2l!k1X}0~^jk4X3t?I{ZSuc&@Tb6wC%)rqWr?@e zd114}yR0*~KofE5w?0M}P>74tj8DMu)$9=GTGr=3T04C5(YU&k<5^f?;js3BXY){Z zTQcxO0;1CLu1q}feVB;LxXy|76ZeuqP@=+EgJ`MJ389A)jYW;FQOPuS;LcV&q;Ic9 zG@YoN{$cvKX8qRs_Ir?1N%>R$JoG$Ol+~q8(`#ha{I^xzVvlMccuM&@A;;k-TvM(d zDGU0w6P+$mev^dnQ^nWLymN;New~s9uQ^egm@2eKd<~sFoxk-t&vLER{=SjVRP{E0 zx9ow~fGM7m&_~FD)^>~CnLU)^J2lCThiDj5XxAsT*6fk(DQQ778fo}mFNoF9n8(g2 z!^NC=9K(Iej*=$wD6F>9bJwUd%kuWO-H!P~a;@~INfVbQ4>)I?n8rc=?u6+Nwt(UFXqpX;8bxC^2@xO*5rHvd43{+KFl z(`fu}_%6ri#z(_e#ioTGI=Y^|v_22k)vxd!d@HHVJz5U5o*sN3%8+L*;|%}4yQs>f zR9z^qw3fI?GD6F=vPmJ%P8d}LDF@{#sH*Fwaz5Ad;eDFNN?y-ptl_mU0MMUkzT<*> zAPBE#1ab+VjMoXw?&}0k9j2^lIJF}8X0B^AKF^wOnKRxt+yR%pwqR^m30K**pyDoE zDKDEgLQbZ7c;k<_1>W}o`NW#M>Q48$1scihAuEwgj$jcJ%~=HUrP$jF_JKLZssv812MC^|L}+h+x89xtBU zuj^~8KVPV4{n_JNaLq7*)4T4A!RxX7c>;Y0_2J+4{$J@hj(jE1xlpq@rhAe(9Ip3e zTGno!&1;ZH9r1Fz<8Xyhlq%`!8_5!(1$i|vE7lWc9Qo(Qr%A)f!BU@yMNW8B9X-|_ zAs7@(nQgsxN!@5Q)jah0gwr{4OT#qGXXSZfq&11Jd0-$8Q0EVw; z0EmMBhu2sDn50)sODX#mFiHM3c33u*At3k+!?In^1-0k0*)8Y7RjZrX0zkxk{v>YW ze?)-PlEUcN95F1|iK2g7FWLinD3#uaX=Q=8l;zkJ!L&eN(}Z!lqIh7=J#l|nCosVS zJupaL1n{A-vBp{$BS~{im$9v-VzRTSZy@?y{r#au}syD4VF3{sXM3BO#mg=Ck*BfaSS#&(JVTwH@xwZN!0%^5n z3wnuIgo^$J2ChG~K*p>R!42u@M#eA=&RD4{m86S3?y_TP<_O~ebBZ^AKds{BH9;renWWG~wAM#v|8fx?b6A5U>javCHo zEGO4gN{r`qZEk_Wu)m1)GYbx6fM_C#JLCIY)5R@o)8N8r>fqP+c|Otg*QXNbB)anH zxz5w+eLmW^)rh`-#*@0)V>a+8l1V4_Y<2$KCS5OXL4#Y{ zq=PIuhD0|SvhFqbBoK(TF{c+tDWF%yOu@-ObN&(HUp_kyi!CzlTY6%X`JO0o;Q!@* zCI{8=G}ng^J1+3Rb*Xz1LQ6#OYP8p((^Hl1ZX3R&AcV*Ulbo_*-*k+4)V@auttl@E zYhe$9c_g+v1FiYa3`4{mRRXqTul0#{pl^0imP!avFI~Seb0d6+OL-2pYr;1sMHj9G z-P0b`leSHbEg^*d!TTTQ9_0M^&{|yPxXOiT6yQ>G96GPa=5q3B;#)JU>F9zSAAo`Q zZ2mnC1EO2C4ICBHLPWKAeU%>_e`3l|DsJP`B<}dH+qdi6fHvD}MZ3%MK(rQzis1)f zrn~7|cT{A+3xiX+Jjy_44i?pDtX&Zbj5Q4LU+?1?(T4_WMYSP)?R#_bXP3<3Wf`|*0_zP%N|vqS@b!_>?^L@>_-XIul5**E5o)nE9@s>;WC!ycrdgXNRe z@mVYFTTIw%H4|@rBKGDYkO}*o@ytt@35yMJuRSmO-2Eq2faKtpSx zSm{9)U9N*(YZ9Hmn4v5K{!e@PpT3`XEDfQT5+X9A?Y3_RXuHPz7qwG50ssI2 diff --git a/test/visual/mpl/graph/references/5_qubit_gate_map.png b/test/visual/mpl/graph/references/5_qubit_gate_map.png index d04aca637ddf420b73d49cee4c7f419e7ae24f15..1bc197fa0556f734c3435295255cdfe6f9bc120d 100644 GIT binary patch literal 5329 zcmZ{I2T&93w?2rVV(2ZjNC`!x1nDh&RDnD7-n09@vwO~iF*m)%K+i)@MMcG6XrOx= zxUZg{moEU)HtK$#fk&H46A14)dtm>LBBAESu-xcEl zyK=oHh^Wnd^_5pJx2w(ewc^ZBb^yg@2HTJK8xvBDu}O=s27a|nsyhYD+Z5PXs#Km&) z-vgIQ(_hEg%;UO+F)ZiE;SSg7QzygNwQR+D#cp>qS^3*d|VKhh4}{T-R%27 zsY(^ph6bCs#AMd~XqDf&1d0hu3{!|~DJD4Jy&gfgKVD(Z=(O)_@VyVTYiL6e7A-L<78>4AEdKxw(+pwBsH+FnhRL+ z)ptdNB!l&~)#W5kMw%3gX(qux+!n>dS%$^^-{yZdMZ~D3*?i_y<{f= zR?lR&ZDX}t_N@hE9IP$Yr}` zyY740^vlbqPW=2)cSD5>b*P>&Yi!!{)ADexn*|fpu?hBddibY*B2PlZoY`Z?KKMal zr@!1YY+2e?i3a72uXk;s)=2?jlB{L=mo+#s9MA`lWYT?B39KH{ z;|dmlcRnQna8%ng0Ec&mOR^?hr~!aqV%h+pp!OZC-fY0@f5}3qMe{!Q4CNH@1)>4p zZL4itD5Fd#8i4aCr~~k{3jkbu8t|mOTn~gzht>lFZ44Kdi3XD0!incriRc|$D5vD* zOu+7H@&s_Bb%>b-?8+>cfqVoRPWJ)(;9StTB76)yfaKjI|MQSse}{l10}`0dmC?UF zfU+4=c^&}!m+amnZ^D$oElLv$0^L+(G;c^rx+&4wU&B`?^XjP>eu$#N?iqN0GI`)K z(?nk{&(EU3Jggcf;s5yLHhd6Wd@6_U*66i_=NSFU5}54lFqMb3tP7YRnsRvR28RFf z$Vnz#%uT1@zRxu*;GB_PUKJN)D$3)o3m0ieCo=N%Fs#1Qu?d@&I_Ta!RB7TeTXQL5 zVO*cnc8T0E6h(yKft7L5f4O6Q#R`~MP)nx#89JygnLpY)v*sw_|LY>+r#|`{877hdWU)3k~^WR zUFG66lTqW-B{JK;$~g1byF9p^&c@I}(7&M|EYbU>tl_>$ z@Kij3#D{Z&hr_i2gEMrp(=76frA%OXY{=1%s@>4_x|=bQ~8EQYOR*>vyhBB9zJi z-&Rlbh!L{=`0(s+DDReath z%g&x(X0Ci_Y4uMJPbL=%^#nol=8`JK9W^_e^{Tih4C>tcVi5VH#Kq__PUFPEn9XT! zuFm0%42l(g2LjpqGcYqAL^Q_*Wl1?VD|TP4q<+pxd^7%O3^qU4g7Ak%+1j`@#_GU1 zS;-E*KI0#+xIWC60vZ@?@_=PDg9b(_kyOPyI_QDA)4 zgjn~1`~zB;Nw1`GNp8C|-U{>clO%!vo~`rq4?!~rbD)!J4bG2nzG}u^8}BEUqxiN_ z1i!0KyzQ|{_pUg=MY_hSa%Fz$?C?|FatN)P@{a0?D7@Z`Uv6E0YqWIfWTn4!moe7; zO}I1(wzN32USP_wE5~@CT;+3BVT7u(;RT_H!4Mv`ExQ--aPo$KLr>2T;Sl#zdZ@QN9daZm|Dh^%^I`)Ha36D zeOZ1{Ue@i(k#`5a6E=JLw0&GCIq-clIxwkBC))AonHxOF&Y3j2Z}}d?_A)}cs3QS3 zAM{r_0ZYTq@xJ2OXTOpyorq;q#me7XMc(vw*?kQ!(XQW?rnfrgnd08Kr$eh(iYm?Q zH(-e7p3-rOGNn4*gH}CIAT)jx=d$y5#3K};{fsH#am+g6c#}KwP^cwAX|n_Sucm|u zK9tp^P7@W+n+02~IDNrUvhJM_q}SAbr}}E)aeJhqH8r0~hm6_pGNH3{lWP3qxH288 z&vhRwzO_rVWUO@Hp5=AD3n@LG-tDq^7jl`2r6Bg`Z@&~(UsKxv4lyAx<(`}0e4*-o z+=@HJ(9tpb-o{QAwW771OWR8o>SopFk%Ve-C%sdsFI;Ys$K^-Y8ZUL^E{_?1h*QS+ z!5GUxYvj1A^!0;oobuZJmb7XrU}2l!k1X}0~^jk4X3t?I{ZSuc&@Tb6wC%)rqWr?@e zd114}yR0*~KofE5w?0M}P>74tj8DMu)$9=GTGr=3T04C5(YU&k<5^f?;js3BXY){Z zTQcxO0;1CLu1q}feVB;LxXy|76ZeuqP@=+EgJ`MJ389A)jYW;FQOPuS;LcV&q;Ic9 zG@YoN{$cvKX8qRs_Ir?1N%>R$JoG$Ol+~q8(`#ha{I^xzVvlMccuM&@A;;k-TvM(d zDGU0w6P+$mev^dnQ^nWLymN;New~s9uQ^egm@2eKd<~sFoxk-t&vLER{=SjVRP{E0 zx9ow~fGM7m&_~FD)^>~CnLU)^J2lCThiDj5XxAsT*6fk(DQQ778fo}mFNoF9n8(g2 z!^NC=9K(Iej*=$wD6F>9bJwUd%kuWO-H!P~a;@~INfVbQ4>)I?n8rc=?u6+Nwt(UFXqpX;8bxC^2@xO*5rHvd43{+KFl z(`fu}_%6ri#z(_e#ioTGI=Y^|v_22k)vxd!d@HHVJz5U5o*sN3%8+L*;|%}4yQs>f zR9z^qw3fI?GD6F=vPmJ%P8d}LDF@{#sH*Fwaz5Ad;eDFNN?y-ptl_mU0MMUkzT<*> zAPBE#1ab+VjMoXw?&}0k9j2^lIJF}8X0B^AKF^wOnKRxt+yR%pwqR^m30K**pyDoE zDKDEgLQbZ7c;k<_1>W}o`NW#M>Q48$1scihAuEwgj$jcJ%~=HUrP$jF_JKLZssv812MC^|L}+h+x89xtBU zuj^~8KVPV4{n_JNaLq7*)4T4A!RxX7c>;Y0_2J+4{$J@hj(jE1xlpq@rhAe(9Ip3e zTGno!&1;ZH9r1Fz<8Xyhlq%`!8_5!(1$i|vE7lWc9Qo(Qr%A)f!BU@yMNW8B9X-|_ zAs7@(nQgsxN!@5Q)jah0gwr{4OT#qGXXSZfq&11Jd0-$8Q0EVw; z0EmMBhu2sDn50)sODX#mFiHM3c33u*At3k+!?In^1-0k0*)8Y7RjZrX0zkxk{v>YW ze?)-PlEUcN95F1|iK2g7FWLinD3#uaX=Q=8l;zkJ!L&eN(}Z!lqIh7=J#l|nCosVS zJupaL1n{A-vBp{$BS~{im$9v-VzRTSZy@?y{r#au}syD4VF3{sXM3BO#mg=Ck*BfaSS#&(JVTwH@xwZN!0%^5n z3wnuIgo^$J2ChG~K*p>R!42u@M#eA=&RD4{m86S3?y_TP<_O~ebBZ^AKds{BH9;renWWG~wAM#v|8fx?b6A5U>javCHo zEGO4gN{r`qZEk_Wu)m1)GYbx6fM_C#JLCIY)5R@o)8N8r>fqP+c|Otg*QXNbB)anH zxz5w+eLmW^)rh`-#*@0)V>a+8l1V4_Y<2$KCS5OXL4#Y{ zq=PIuhD0|SvhFqbBoK(TF{c+tDWF%yOu@-ObN&(HUp_kyi!CzlTY6%X`JO0o;Q!@* zCI{8=G}ng^J1+3Rb*Xz1LQ6#OYP8p((^Hl1ZX3R&AcV*Ulbo_*-*k+4)V@auttl@E zYhe$9c_g+v1FiYa3`4{mRRXqTul0#{pl^0imP!avFI~Seb0d6+OL-2pYr;1sMHj9G z-P0b`leSHbEg^*d!TTTQ9_0M^&{|yPxXOiT6yQ>G96GPa=5q3B;#)JU>F9zSAAo`Q zZ2mnC1EO2C4ICBHLPWKAeU%>_e`3l|DsJP`B<}dH+qdi6fHvD}MZ3%MK(rQzis1)f zrn~7|cT{A+3xiX+Jjy_44i?pDtX&Zbj5Q4LU+?1?(T4_WMYSP)?R#_bXP3<3Wf`|*0_zP%N|vqS@b!_>?^L@>_-XIul5**E5o)nE9@s>;WC!ycrdgXNRe z@mVYFTTIw%H4|@rBKGDYkO}*o@ytt@35yMJuRSmO-2Eq2faKtpSx zSm{9)U9N*(YZ9Hmn4v5K{!e@PpT3`XEDfQT5+X9A?Y3_RXuHPz7qwG50ssI2 delta 1180 zcmV;N1Y`TrDW3@;iBL{Q4GJ0x0000DNk~Le0000i0000i2m$~A0Fma+R*@kTe*{!X zL_t(oh3%M2Y#T)w$A7bqlQ?AE=0R~%sU?k@D2+r?DRE5bfrp?f#8pL$RP}!2-A& zd94F?BhD&T*9k2$^oX$;i4mK^(Q5jyP-nh1rVz1JKK+(hl5~Ls`n>qO6vlynim!i= ztXYQR1;BH zr5J+&xhHyBsdfO|`OS!U=XZ1+MeyoN@|iS^yMCha(jW%>aw|aRDS+e#6CvZPXq?9p zfV0h}!_gTClmL-PGBPP~#*AKc0X&Y#lT|PbP^+Rkq{|thvi(m)Gad7VRYIGVl0SRR;=>)^3b)mwZeMp&X zR%$EzP-&NU0MMBU8@)acVmT(MjS@QtI}oO}SV@>eMDyPnw2{g_l$+Fld#-&wCGW7U z<$SNogIM1zHu6bsf6L9XuhU`dX(q!&dpO>$u4kj``6KcU^||)QO6gV$AROfuz5k8? zG`SG6IBsqLSV>4SfkBn|6Xih^skRx7&jM6c(0=|*EgH%42ho4@8sn`eUIN7ShJytJL*lWTde{8QW>DnOe^WZBm{jPNxKB1-ifcLn0xL|io!*&JPXJ(=7lbrUB)0h zh-?b(2F`bx@X1>hZumG_eK;I8g&LLXeWt(5rjw{u^j|pyjts#^mrc)nSpEhkl6*D5 zYy_?3ztG)M6xobR%}v?%pzAR1>Nz=H{N z&~z}n8UP^>g5GxC>9($jdxOa6$r^!a#)Hg+cTSYsvX^oCI31_(v=lq*9u#6*JJ&^< u0FH9%k5H%XGrQ~);&Ao9xV?J@k<-6yLVKK9rMKPy0000 Date: Thu, 26 Oct 2023 23:23:44 -0400 Subject: [PATCH 6/7] revert purposeful failure --- test/python/transpiler/test_dense_layout.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/python/transpiler/test_dense_layout.py b/test/python/transpiler/test_dense_layout.py index 57ea5bdef4b4..669fd485b358 100644 --- a/test/python/transpiler/test_dense_layout.py +++ b/test/python/transpiler/test_dense_layout.py @@ -66,7 +66,7 @@ def test_finds_densest_component(self): pass_(circuit) layout = pass_.property_set["layout"] used_qubits = {layout[q] for q in circuit.qubits} - self.assertEqual(0, set(middle_qubits)) + self.assertEqual(used_qubits, set(middle_qubits)) def test_5q_circuit_20q_coupling(self): """Test finds dense 5q corner in 20q coupling map.""" From 36112b4dfa13e4ac5cfa05f3f1281f18b803b523 Mon Sep 17 00:00:00 2001 From: AngeloDanducci Date: Fri, 27 Oct 2023 14:25:42 -0400 Subject: [PATCH 7/7] tidy up trailing whitespace --- .azure/test-linux.yml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/.azure/test-linux.yml b/.azure/test-linux.yml index 1934a6f2929a..3268f29bd4f0 100644 --- a/.azure/test-linux.yml +++ b/.azure/test-linux.yml @@ -207,7 +207,7 @@ jobs: # a new internal interface that will be the default in jupyter-core 6.x. # This variable should become redundant on release of jupyter-core 6. JUPYTER_PLATFORM_DIRS: 1 - + - task: ArchiveFiles@2 displayName: Archive visual test failure diffs inputs: @@ -216,7 +216,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/visual_test_failures.tar.gz' verbose: true - condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) - task: ArchiveFiles@2 displayName: Archive circuit results @@ -225,7 +225,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/circuit_results.tar.gz' verbose: true - condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) - task: ArchiveFiles@2 displayName: Archive graph results @@ -234,7 +234,7 @@ jobs: archiveType: tar archiveFile: '$(Build.ArtifactStagingDirectory)/graph_results.tar.gz' verbose: true - condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) - task: PublishBuildArtifacts@1 displayName: 'Publish image test failure diffs' @@ -243,4 +243,4 @@ jobs: artifactName: 'image_test_failure_img_diffs' Parallel: true ParallelCount: 8 - condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true')) + condition: and(failed(), eq(variables.HAVE_VISUAL_TESTS_RUN, 'true'))