From b590d973924f2fa5718f87b23282914c979b95f9 Mon Sep 17 00:00:00 2001
From: Samuel Lopez <85613111+Samuelopez-ansys@users.noreply.github.com>
Date: Wed, 27 Nov 2024 14:32:20 +0100
Subject: [PATCH] FIX: Edit sources from file in HFSS 3DLayout inversed order
 (#5490)

---
 src/ansys/aedt/core/hfss3dlayout.py              |   3 +--
 .../workflows/hfss3dlayout/images/large/push.png | Bin 0 -> 1987 bytes
 tests/system/general/test_41_3dlayout_modeler.py |   2 ++
 3 files changed, 3 insertions(+), 2 deletions(-)
 create mode 100644 src/ansys/aedt/core/workflows/hfss3dlayout/images/large/push.png

diff --git a/src/ansys/aedt/core/hfss3dlayout.py b/src/ansys/aedt/core/hfss3dlayout.py
index ca3b0b85346..307ffd1f82a 100644
--- a/src/ansys/aedt/core/hfss3dlayout.py
+++ b/src/ansys/aedt/core/hfss3dlayout.py
@@ -2292,13 +2292,12 @@ def find_scale(data, header_line):
                 self.oboundary.EditExcitations(
                     [
                         "NAME:Excitations",
-                        [source, f"pwl({ds_name_mag}, Freq)", f"pwl({ds_name_phase}, Freq)"],
+                        [source, f"pwl({ds_name_phase}, Freq)", f"pwl({ds_name_mag}, Freq)"],
                     ],
                     ["NAME:Terminations", [source, False, str(impedance) + "ohm", "0ohm"]],
                     ",".join(str_val),
                     [],
                 )
-
                 self.logger.info("Source Excitation updated with Dataset.")
                 return True
         self.logger.error("Port not found.")
diff --git a/src/ansys/aedt/core/workflows/hfss3dlayout/images/large/push.png b/src/ansys/aedt/core/workflows/hfss3dlayout/images/large/push.png
new file mode 100644
index 0000000000000000000000000000000000000000..2eb7c639efbcd451615c5c23b2e8d40d9207fc18
GIT binary patch
literal 1987
zcmV;!2R!(RP)<h;3K|Lk000e1NJLTq001BW001Be1^@s6b9#F800001b5ch_0Itp)
z=>Px#1ZP1_K>z@;j|==^1poj532;bRa{vG&=l}pD=mCOb1snhX2Utl&K~z{ry;s+k
z+c*;S|Nqs#?1Sy;b}L7UIcG756v4Z<D6?DB$K#io9qQBpQ78fzmGuSx!lRVJa=FB6
zwSwa~xLhtc91hs;_m}`=Sq9g2QPnl-JV#v=sO9}1qN=K<Lr<p@f*^otn$UH9;!ObV
z_dCk6#PxPVe>|htbf`g1{9zcTvG4n7oOv^Urss8`&objU%2CrelRWPx5aF5tly0Ci
zblk>nQk@9ODC;uLyqQ1K^SaPynR)%0C+`!0f+`o=)NM-u4s7Ea-qaH6{`gRC@OL#Z
zuRrtTeFAXf&BYJIUZWkrvc6-hc9Xz_o=69R83&F(c@xo0GM_hny#HAnpVyyx^8RiD
z#X~Ck!RUpslMOVbp;hfeWmSWAXmR5lBZ0_|CX@}vpEQt^B7dTXdd14~T*-mLBXj1-
zWCC!#UMKGq?^mu5t31H$brh<E>h4f4YwX)KPU-}`QMd&m)WyU`ZDF+)j^rsiZ(2Gy
zQgf%PLY!9^5A=L>=3YLwH9?v-I?usQZD7=Qxb063i0b1cL7nBe`vl<qKzMVIs2r-S
z(I1Y`@;82H;*WAfp}T0#8v<Z#w<-1sRk}XHjW>83w|J`uhINWzx5d6&VA)w%<s+Jd
zf!x|+IWDoNt}qDn!vv%%PSC}S)9?*d8{k`)z_JSro&h&mVL5m>R2>pMMDA{2-VS)_
zLpVW$>f*u=7X)Qv-PQ1o1dV-yQ>-cDLeDzn{4VmAI1d-R-E)|G?*X7`8bnEqIwB=5
z0W6poB77A9MvfwP5ab7JZyBtLn|oJ;`5u1l@OCM}s6rBD@UkVeF306|r3Ol*xrwkm
zV|B~n<u$^vf}gEnw-I*t0>&x$01(D88b8D}v3c7KvLwa2j}bZr+}cBIEwJbc_=OMG
ztEZbqiH|>q0I40qWE6$LE6#NZz4Z|2Il<~kgEPM06GWj0hXB0v4Q`%cGx#{~-47Fp
z$JTVLvW<D$9pb2<noOixh)W-#u($YeD-dL7ICcfI+9C5Iyo^3d%ckSyi2WCwiz~EN
zLrehL%|hwu_&%gaeGLXR<)p6&!W=7QA<%vTkUWqu3L4!;KGq0Q0`N+zPD6U}i8e%G
z?yzb~YHADHtDx08q*j7g<#J8Ox-NLg0$N(Zs15{>V%u9Nm{niz0$_)8O3md<1T@d0
zGZ=9WzR2M*)o;2s7FmQ$6-e?ELaSuxoRF9v4+-eKCv2|-5G`QxCg<iza4}YQkB6M_
z)@JmBEpJ)VyElmK9E*V$(oOnFhRj*w$k23(9rrG<;pIBt#uv}?rfMb6*TlhMJfo>R
z+P=c6JW?|RFh(eCgL67x_yRM%;@IiPc*`?mymU@@p;ILZV?L;17b||3F(h4-yugu8
zn7MD*^Bk)>BQQ#uNk<b=1Mg*%$;ik{{I@@$YeV>E0khEHl^(v)VG75Cm6iyiSLjtq
z01l!{3X_qLd3${CGsLM2$FJaK3xsJ*&3Ob#2fd~==___^35)VR0SJ&XB>=g(lwyzl
zKz42MpZ0{FDl@eRk$<8+&)CrW<CyAC#ETaUvkYfCq)$JI3zs2zz`A9ql?=0hlyBfQ
z0k#auFrZADGDO$Z1TRHDa@{MU@`9An^&SOE4alz*elQ9~z8g-4*4A-WOg1+kLEO^W
zB5t@~Mr5+-ukg&2$>uU`{>o%yTsx#jjKp8V=r!_UhHQn#^58MKt*Mdyt)&TKT-^i0
z+I#?z$|Qau<TRYKNL+RXMwaIFxT1skG><~F(HPIWDc!D`J8Yk|gRPa;FtzpuwoKEd
z=HO~>u^rb;(*?AMeZ@|r`PTHW6PBzAp&g++ZQ%BFLIIGl4oI8SHuPW$)ct@<+Cbep
zG(!Ws>F`=A)`W-MHfn_(=T=6U)d8pBh-1(6%G`c4Ec6#03$<WUYOyCzt#)urjU|4A
zH9;m}3bi>=X7gbJsn=~y4cu64`VnR;<y2}Xt7Wfnt_D`mM~#=UXK!$4?N`!V&}8?4
zO^3{-?A|#aSw&fP?^24T8R7a}^(@;zdkS7sx~>U;)NV<~Hv;eoAigPhES2I5kD5Qw
zE^H|HE;QBi2@I>C93V{y51r@lGO{-F<WS1Zmk^#*zzM*Vh-iYYt*3)+n!#_RYx!uQ
z{;w^WJYCnJYbwOu2(20sQ44~0ZX!ik#)k<+1OH%CDvtmV6NvqH*AaOq0v{WGKHxL`
zZe7;?YW+_pkSGv`d&v~8Y{mXH69|%)EqOx?+~_=CQ+n#;fDhF3NAspL`g#6&UFfsS
zy#CCS_jeNr!kPf2FK8HIv-&^VY#x8bf2L=dzmk>r34kOb!!RaOR4huO3F(xDBs<C|
zGCBV@&b*l~B!6t8h!s=*-u>h~%gBEY`Tm(NBt>unAnHh2Fl^g?NB&HLR@1a;lV7dN
z#@p?78h^F!v(uJ8rn4EO2N9>2eZR~?^Yi@SGtJNM*5vu!y1dK5mSs)<j+>^z{{UIo
VT`|B^wXXmG002ovPDHLkV1mLSy5|4@

literal 0
HcmV?d00001

diff --git a/tests/system/general/test_41_3dlayout_modeler.py b/tests/system/general/test_41_3dlayout_modeler.py
index d05fafd4e00..cc4260d976b 100644
--- a/tests/system/general/test_41_3dlayout_modeler.py
+++ b/tests/system/general/test_41_3dlayout_modeler.py
@@ -336,6 +336,7 @@ def test_13a_create_edge_port(self):
         assert self.aedtapp.create_edge_port("line1", 3, False)
         assert len(self.aedtapp.excitations) > 0
         time_domain = os.path.join(TESTS_GENERAL_PATH, "example_models", test_subfolder, "Sinusoidal.csv")
+        assert self.aedtapp.boundaries[0].object_properties.props["Magnitude"] == "1V"
         assert self.aedtapp.edit_source_from_file(
             source=port_wave.name,
             input_file=time_domain,
@@ -344,6 +345,7 @@ def test_13a_create_edge_port(self):
             y_scale=1e-3,
             data_format="Voltage",
         )
+        assert self.aedtapp.boundaries[0].object_properties.props["Magnitude"] != "1V"
         self.aedtapp.boundaries[0].object_properties.props["Boundary Type"] = "PEC"
         assert list(self.aedtapp.oboundary.GetAllBoundariesList())[0] == self.aedtapp.boundaries[0].name