From cb45133e3b47b9b53af4ac671e2cf8f8226bdb10 Mon Sep 17 00:00:00 2001 From: Nathan Mannall Date: Mon, 17 Mar 2025 11:52:40 +0000 Subject: [PATCH] Add test for fractal box - Fractal boxes will build without error using the parallel build process, however they are not correct. Each rank builds its own part of the fractal box however there is no continuity across MPI boundaries --- .../user_objects/cmds_geometry/fractal_box.py | 8 +++++++- .../fractal_box_full.h5 | Bin 0 -> 49648 bytes .../fractal_box_geometry/fractal_box_full.in | 13 +++++++++++++ reframe_tests/tests/test_geometry.py | 14 ++++++++++++++ 4 files changed, 34 insertions(+), 1 deletion(-) create mode 100644 reframe_tests/regression_checks/TestFractalBoxGeometry_89b5811d/fractal_box_full.h5 create mode 100644 reframe_tests/tests/src/geometry_tests/fractal_box_geometry/fractal_box_full.in diff --git a/gprMax/user_objects/cmds_geometry/fractal_box.py b/gprMax/user_objects/cmds_geometry/fractal_box.py index e78be057..48a22dbf 100644 --- a/gprMax/user_objects/cmds_geometry/fractal_box.py +++ b/gprMax/user_objects/cmds_geometry/fractal_box.py @@ -110,7 +110,13 @@ class FractalBox(RotatableMixin, GeometryUserObject): p3 = uip.round_to_grid_static_point(p1) p4 = uip.round_to_grid_static_point(p2) - _, p1, p2 = uip.check_box_points(p1, p2, self.__str__()) + grid_contains_fractal_box, p1, p2 = uip.check_box_points(p1, p2, self.__str__()) + + # Exit early if none of the fractal box is in this grid as there + # is nothing else to do. + if not grid_contains_fractal_box: + return + xs, ys, zs = p1 xf, yf, zf = p2 diff --git a/reframe_tests/regression_checks/TestFractalBoxGeometry_89b5811d/fractal_box_full.h5 b/reframe_tests/regression_checks/TestFractalBoxGeometry_89b5811d/fractal_box_full.h5 new file mode 100644 index 0000000000000000000000000000000000000000..985f9b43b1b9fb6546d67f4b88beb20e7159296c GIT binary patch literal 49648 zcmeFa2UHZ#wl6&7C^?Gcq#!xNboC%0h$tY*EFdT-NrE{Hpd`sT=bXbZgz4@Yz*eubHBUZ_tyRHu@~LDYgg5-s$cD0HB-B4h_|y|xrlc$FX_j{ z%}rt_3H^LsczpkTz3KbKSqLx0d;Wf5`Ay?~(^}SFFDxW}(vLW)-+ZCp@)g*AGk#Oo zbv8Dn`IHOt7wGR)B!$Jl-nbVSf9Ssz0(Lf*PCpvF{&%Yv^l9+7o-Ke87P>v`$9w4h zpX8_R{RmTjJpK!p@Na$nt6uZ}mdHZlB8ic>NjpPA*LdwGk)&Bze&+aLMZav~m&PvG z2iK1}{8r*0yuZoe{3*xPKRm#9p8v1pg#MkJ9}w`9On-3Z%jBDX=O(QR_YL(5_YV&G z1p`TFzRX|6QhulVNl22MzbHTHC%p6ddFJy=k$&J9?i=zuH|b}7_Mb@=em+^|+xg@A ziy8mq{r!WVWcM@3GGF=+x%2(kdN{`}QGf9U^@%kOvmuMp{nUg`aKU#QnF?O2HYe@}nl@b~pw zm|r}98XD<$uaZcAt=}(On74`F=@v>}sMjwoTj2iJ=|X)*h3d0}?G^8-xqPTwHk;D|7VuwefHg@`c!pq&a{3Sr^F+x@)4!u_`^`1pr~hx&U* z{1}K7f_Kb^eFJ>A&$G5GgoFl%_=bl2`-Um(^xx|nq~LvEK5wA^{vSdDgMEAh6n2CL z2P!!Eh6MNrh3)qL)q)>w{&D@pVV>H}%Q)!o6=dxn68vM}{57Whu;*Xby8TN{b(jDA zTlSmvlL!g^edy2o9|Zm&@CSiE2>e0d4+4J>_=CV71pXlK2Z28b{6XMd(S@9`pCY(84;=U)Pxy^hDwJy5B;&zpfYlwVeOy z;a_$}`iK6l|LN`j)4qR>z%M)d;U)QXG5=yeeloxKPye!>{?DcU`^7H!C;p1pU;R5y z!D0SCR;3pZ`iK6Pf8ggjB-U(u?_ehSSNgB!(^{2ZB`);yLivB_zXt+;^+PSptEr0r zd;3H8|8$f6AM}&4|3CW4-l+a6ckp-d|4M&VD8CZw{!agg{CgqrSN;99>wrXZ`aP~K zpkedNy-cz>@QY*f+mmkhn{W4c5g~Uf2ThJ4f7a z`iJ~M;12?S5cq?@9|Zn20@5TBb^ap{bu9~ts>nv7@^F!;+XP9}>k1^Qr45P7n?|Dc zX|Yhxgs@U?aB)!WTzIJ$q{XS@uQjN*XWglDds?U(`nr)b`mK?hBXpwLzBEO}UDS*& zc~=|V5~_OKWMkQJEBcZXs#!TFbU8(0c1b73Sl{J{tsglWE1B>%_5rU?-19v*;x^o} zi{I+Yh@Vu|PM{c;C74x;ojem0chcGxFMwykNR_I+vF)z#BII8)Qb?LVb|iE+s|U`Na7 zl$Oq{OpMHwiI~VNjx))cuFT6S)*xkziMVB39d6C$X%NogC<)GK_C1w@`6=Y?PKnA5 z8oHXhMNczt@KJo;ZlBwEsY~?ozZoRuU+|jD-`=QKFlCfjP<--MLExR`g;Tt-g{y6^ z6xM4f7xipATvQp;U$i4cym(~Pr?{k|xj5=Ndx?tQsuCXE+!9UXQOP6m<)tzG)Y1$A zW+~rkfwFt)E@gu&3d`a|9+%l)TUyTbd2hLjD6Ks4E^CEH2~wdu6<1NMa=K#U-bI!B zx7k;kJET?)DhyW^dI(qDb6Qj7mzrAD_U3#QD@mX_LwaR(3NyYs?K4sB_?fL{!-P@I z=b-~NQalYcmdBpepaf$ z);C$1)E8LqsqgkEuD@k^slG*zt6|N3od&4t)8H1L(QwO>Xb>;|(6IUPlE!=c>>2}O zk2TtEFK=`l9&UWk!`jqwTe*q+bWKw@DY7Y1r>u#2eXvO?VYX>9cX4wbC)%95eRuPo zvef2nXj^k%*;w-wCwoiCF@=`;>=i9%SMP4=zMs_M-%#I@-+!THmUFtr#zLf3Qd+x} z&RpAS|02A#pO)JCHNC!dFk`Tl$Noht(a6)b2vcYqHnC_sma?Hu3LI_=c$C&A=v&*S zIn~#;`_`Sd;H2+uB}O9c58tb`8=;`xc;ou^fy9V*d6ARtkIahNU3Rs#n_fNJ&ZwDe zKQ8yBea3WA$Kp4#9T(knJ2oa+ci3VZJNEI0b*NpA>F`~j)p19^s$=Q^y(8N0T!%K} zb_e?AeaCw?PMVsCC~d+}k>-3%mqy~Wq!ry+N3&AcK|A)~AkC;Oj<&rqlO|kVM*Eo7 zLc5Si(6(1zq;tf|x7xwn%;Xt47-aieov;NwoG<)1ofG3@kyVFCI^ zODVdtf-*g%N1JXdZ9xxku%icgy3^aY`_d2jM$kzX(R5RnMEY^|YMGu_xT~W~rAsqSr%OlQqU%VCZC5DG zwM($cyNf+$PuHeuYF8ase3$R*w5|y6f-dcVs;?Yh4O8_qsyA zPIbxYfA9KG%hj!HB-GtzE!{1CSE*ZrU#r{bno0L-)T;Z~ddKc0_08R&%fA~H+28%R zEV_HF;bgalR%W;7@}h2+*6MB}W@~rmc1E{n`I+vQ$FFqHh~DjPUH`J1wt22wAd9`H z-K&}r?2Vi-?0pgmdVMPGdNXZZdR_Kx?d?(D)hn}SZ!b@MWN+{8 zxL$+ElwS4M*}XyZ;$DtB)xF~WExj){b@#F|`g*@64D~jST<=|!Hrc!O@{8W#>`%SP z?^zh%wsSLR@`8-PWfF{l6S9o=MyiZ+K#Os(!idqa6EHxEErVOmnISK=i9w6>Wk_!e zVVt5JV$=>qGx}e}Gn6)^GMoi+7|(f%86{3t45LSljM(!uMl2i7u*f*g=s$ds;WRMH z(A{;9F&O@Wk@a|*k?lpo@0ZU<$rkBNKl&lZ!_@DZ-sTmg8({wK&(l zCVcc+JO1=&H{NPY;AO(6@TEfM@echFyzamaJmk?W+<)%_{O$7RxHr#RT#$bTw>A2X z*RNnBz&B3f*f1|~vQ>aMP$NQkwM!6gmlqSh?D9n6YGs18MxDr$SWbB6>k_W|Muh(z zGa|hh5WS&R#JP=jM49^@))a4mpvbPFX4pY9_{l8zAH z%OZ)!6UT|NJ+VZMZvt`5D~WjSlS)(tXApjo*~F8)JmPY9A#wF~36aHJL3Eo|6E_3v zi2RyH;^d z!PdhpQ0oyZbYIC1J4!g<@EcCJ+>9H#?dO5RO}tQioDa6l@nsK7=4MhxkJr{%1(c(~jkpv7pAptF4N<>}keq(vl`n(%G~Ge?+cpwgHdYefvxs${$vnYZ? zCzN=)XB%;X>_D)jni8dfD#V!^g2bD=X}rDSJig^(F&=E;gXe1U;JTxF*ugMc@`Zk3 zWLz%{=@M)}j48Jf2N7FvnAHcIQVR9>j_|xALHYvqGqmofH~YAdfzItA;w-YopsMjL@;qWOVB~YqYFt zHEQ=^9r}s95nX%K3w=fRLxUa#qk-J}&`7zXs6s{*swfzPCP&4ituGVN5rD?H_$IDM^OW@F*NPe7<$fQ9L*jcM|)K#(7A0BsM6sHlO5K_ zJcurSeHy*pG=SPW_n~l%KufJL)FZ10bsM6iB2PL{zGtoIz~v^iudE);*ieIBn5{tB zqf60==|c4Nx;*q*einM*Vj8;UbrLH5IUdcKK7mHwiA41S51}?+L(#KAyHLjgFErxI zdi0iz6RM_ci_VDxl=F=tI@hg<#)l}O{!-GYbGsn=4dp~#VrGH-p~qmt>niw+Fu|Ni z6L5H#2YfnXz&6pHz~CGO@MkOrYp0$f+dmf~@-0e8pX>??)f^sNOD_dqB|45nePQBU zk`XbOxq-N0d4$NfO(lvLWrWdOJJCWqMeM$CjhNW@h={72CcaIu!a0H$W*-xQn9O46 zm97Yn-(L##_;leBJ`)&sAAv_wtYN0`YAE9846V{VV06kBXtHfPyiD?km%@TzcUKrx zy|5p4wH|><`jId?JQ^w#pMa}dV&U7Gc*q@h62>?t!N70HurD(eCWxfNv3(g3p2~z9 zX0zaccn(xtoC}LM^PtboJXo8X4@*D+JTY7Vb8HHsOidwle^&^z)r;U#+akElxd>*g zEP^ugVVC!Xu%KZ+-l-72xK{v~o&}IHoDWCU^WnRYJjhm;3lELtz~Lv^FlZ_Zjy}nR zM=oc;vBq@heJ~AXny11iuae<-W)gg$lL(_>0@SyThXF(^Y|xH@yRwc$h4)d=)-Dn@ zY8->{orj>9{(fj!77nkl?SWS}213aie|Y$rFTAPb4Fz1c!e>V|LZr+cjuPwO(b2W= z(+hjZ@^vNj;Io3wk|^v^vVf{uCa}oF01m9sfhLZcFmsDK3=C3+u2J&vN}dc1>Xd{v z*G1vyIYBt7$`8-Fb3^%LcDVKe35F=o5n=c{!hPopqF?bL;r{9tQGNb8LFv6nRAOfd zhbv5?;d2kM)U1P0IoU{jdRs-j4J;uxe9t98U^*Vo?BY7omAF6^!er3g9ZaRfcupP&;?(krH-M07c`*_`=wAYI)!`(o> z$nuc9YoHD| z-fMuc9Bp8%s1H(7jR5w@6wE4+!I2dxh_qh;OwDY-H32(tV#ppG2wnqj&pCpLz;z&h z&=uJ5c>t2}2JqHy6EL;j0%p~>0hbS+z^~35xY_yui;KR%6zl}MbNs-&aewfPV>bwq z2>{iL13?2@5Gc421hO-N0W=Q*Mi)ZBV!J&cwRsOz-|3cw`IcRNM@9_-q8`dFw$c(;b|ea0Mn$ z*MT+9oPhtGwcyoR2Vhlg4@M5yfiEjo0zFo1Fy3VeJbh50$BlrV3UlCNVhYL!jKHLw z0Z2Nl3nVDo;AOieIH<4;=*6pnW3QCKl{JbWuU-yN*p~oHQ7?Vy_VGp(lvf{GfB(;WlC=HHJ`L zUP0u}4I>9z29cK~0|;jUfvhj|8x$l1#+$g?l?h@(+85`3r(>Ah5l6zS(8M=LWB zrfLcT=?Tacj~HYPPZTnjeF$+y!Vn$K-3Z4!Z^VXW6XId$f@Ggujks}HBe!eF$nHQR z#Av-XveQo;Q7TqIj?PLW@drf_JHMENr*)Ssja(y(XP+ZS zm_l-^awplqqk+8XN*Vc3W-fWWFooRkJcdk}J4|k^4j^9{+f43DUPGpy0c6&rx@3(G zCGzlgadLAKH<@S4jKxgsq=kRZIg4X49TuydiY@x5;w|Vfz~Ym!t;H8}WeeXvl10oh zXfEkNHNWzd%Un#o(v;!Y(bIl*MX!5QMz5#!KrfSc(<{Lyz^KbnVysd(W~gFz3`@_A zjCk4IjB{TPGF&-MFs`a4GK8O|Gpg6*F=RHEFeW9d7!D`u83na1j87+MjNOVojH#nI zV{3UIV{gi7#`vmpj78TkFb)}xFoJhqWk?>m!8j2x&S)mzV(fZzhY^u*kFk5vL&o_- zj~VYrpE5L5UNBO^1Moeb#Atfbtrc!TS;s++sLpodv$mz`_kAY95;n? zIZh~va=tv9#2F>b&7~n2#Wh~`iOXkr7q{)UA4&cFn!UoLn;DNZ5aZUvh0E{59}7)DRf!z(F;={_vvyW zI}t(Q(A5WpTNEdReRd)utv4$~AhJl5S`aGgP;g1Klub|Un|HR@!@_T3a;MzI_eIgf zOSB{!I+x-sEn`|I&D_Slcwo_{ z#ZO#X7T+4@lPOrgRpy*ptIT6Dz9lQ~Z(L$o+puItgHv`Fm$R(&s&ZM1+Gp9)R!h0S zdzo@Vau4MW>1fN}PCYJfDR)lZMo>!Ojkd4CUDZZ~%KIe6`G7>3@WUXt89v7D= z33BdKGODOk3U`@NV%IWK{?;F@{E-JL8!qBiNqw+VC1xa9B_ZsBiu!$F)tOjF)eCAF zsv4Xls;e?Z)Qn!PQCp;+sy5+tPAyG}UwzJNg?evrjC!M2k2)OtroOCPd#Q1L&{Fou z@};|*ZY_Q4CBE!w>8fQ1>SC7(ve1{!guGwIbXL+he91#Yo+m}4kq6fZmYmgiTB59p zy1Hxbnm(y1M4@ZmFL|j6mPs#P-fpv8I^^hb&J`8Q4?G@Ou4T=kWp#3y)($ZbtyJzf zt$^$XEt@N&S{3WL<{jG0v=tRyv@2FeYuk;LYv0>-PJ79o8SR2$DILo&3!MyCU!6No zPU>`kdY!1ki#ngh=5zwyOXxC~M!KOk8+B`YqI7k23UwnCdUbE0w{>$g+4K}uW%TO# zP4wRE@zBGj59`rXGxf}6oAjhc>RZ&-LVWxb-XQW%XkejP;eDJL^mM@6o>(ouL1s zuvmW==+t*?yr|Er`$FF>k;8!4B56P}*EBF(XKC>;!#=R_m@BuD<@*Fk|wwp@A)r(Z(am48`W7-#6M{^xo*59Gmea(M85m-IB)E zlFG)hh>r1n6SA>}n7y&&z4gX!&3?woi3g0~nG?p}yi$z0%kzw5(khJyEn1C78!_Yk zZ_gQ%CC7~8mp?JCrF=4erpInl=Ca5nUqI62cz~iw#lGbx6XvESEq&G|o21s6TwLvG z0{w$c+NsA(Se@cbzV1#ldAOv&fq>yjzna?4cO`nl;)#f&NID>k#0vwUWghsDg+ z9hEf;W>+`!lF&12XIPloeOYN1GP%y|Zpb#XZhC-OD&wG8&#B{P;aQ1hN3Um^l_eCJ zz3QztV+6FCwO99=4F#VzbGkHQ_VB_jv#5|~W-Oie zqTD=lV}p5SB+Z;GOPJr%A2k1d_KNw=`?t-7;+~ls!D;iQwk#F`-?%J>xCAZO{3I+6 zD#=;MSg2V{cWPU#$T6|-dO)$bU2JdRf6>+A>E3M?JBoH$DB13{I8BbUkSmI{IBuJ4 zF$c0Niem~bY85LjfJ)Z>>drnih+6jVO2DBb+p!X;tWLM)G! zyt$E^yoE)O9G@>v_ByzPe6T{9d`wD{ta(eHyzZ?zd3>W4nIYpq<~DI7KW^DdPK);= zZ$2AJZVfm>jseHXY~SL^RUcExP8!+dclm|nSo;dHxJ5m=C8&)Y{-m3nQo|&hwGEQH zI4_eo)Q^)77vCqJeDs3+C32d)Ztr(;@_lyXuqiKMLkJ?)5#q@BRvBdXVMS!eC3VDN zgErzRV}!U$l99Z%R!CRBJz^2D4(Z;x5g91(MrK5IBOfn@A=@t>M&2%pMnuM9kr!tZ z5w|;Oh~|mnD z=mUgh_&HLP^bX00nnO0Zvw(mY4)EYI4^TG~0J)e5$lWXnVx*S<_+9}Ny-)@FSTsS2 zkuKN~X9T3?%)$2r3fN|`5**-j0I{E)K|kMmu#vnK97*y4PTzNd>9jo{#%4c|SbPjj za76>dj2IxLlK`kUlfcg0G=TeNfiSB)5Uy7Qs&vZ0Q=|$k-dqQS6PiHj**1_VMhCQz zUeI@w0DIjAK*H^_pfT(MI3al%bPrw!cVi|%n)6+-Y3W07n)exSnt28Gym