FIX: COM connection (#7465)

Co-authored-by: pyansys-ci-bot <92810346+pyansys-ci-bot@users.noreply.github.com>
Co-authored-by: Sébastien Morais <146729917+SMoraisAnsys@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Massimo Capodiferro <77293250+maxcapodi78@users.noreply.github.com>
Co-authored-by: Samuel Lopez <85613111+Samuelopez-ansys@users.noreply.github.com>

Author: 6 people

doc/changelog.d/7465.fixed.md

@@ -0,0 +1 @@
+COM connection

src/ansys/aedt/core/desktop.py

@@ -674,9 +674,14 @@ def __init__(
 
         self.logger.info(f"AEDT version {self.aedt_version_id}{' Student' if student_version else ''}.")
 
+        # Determine the starting mode (grpc, com, or console) based on environment,
+        # user preferences, and whether we are connecting to an existing session.
+        self.check_starting_mode()
+
         if aedt_versions.is_pyaedt_in_edt():
             self.logger.info("PyAEDT launched from AEDT installation folder. Forcing grpc mode.")
             self.__starting_mode = "grpc"
+
         # Starting AEDT
         if "console" in self.__starting_mode:  # pragma no cover
             # technically not a startup mode, we have just to load oDesktop
@@ -691,6 +696,8 @@ def __init__(
             settings.aedt_version = self.aedt_version_id
             if self.__starting_mode == "com":  # pragma no cover
                 self.logger.info("Launching PyAEDT with CPython and PythonNET.")
+                self.is_grpc_api = False
+                self.new_desktop = new_desktop
                 self.__init_dotnet()
             elif self.__starting_mode == "grpc":
                 result = self.__init_grpc()

tests/system/filter_solutions/test_desktop_types.py

@@ -29,84 +29,77 @@
 from ansys.aedt.core import Hfss3dLayout
 from ansys.aedt.core.generic.settings import is_linux
 from tests.conftest import DESKTOP_VERSION
-from tests.conftest import USE_GRPC
 
 
 @pytest.mark.skipif(is_linux, reason="FilterSolutions API is not supported on Linux.")
-@pytest.mark.skipif(
-    DESKTOP_VERSION < "2026.1" or USE_GRPC, reason="Skipped on versions earlier than 2026.1 or gRPC mode"
-)
+@pytest.mark.skipif(DESKTOP_VERSION < "2026.1", reason="Skipped on versions earlier than 2026.1")
 class TestClass:
     def test_lumped_exported_desktop(self, lumped_design):
         schem_name = lumped_design.export_to_aedt.schematic_name
         schem_name_length = len(schem_name)
-        app = lumped_design.export_to_aedt.export_design()
         circuit = lumped_design.export_to_aedt.export_design()
         assert isinstance(circuit, Circuit)
         assert circuit.project_name.split()[0][:schem_name_length] == schem_name
         assert circuit.design_name == schem_name
         assert next(iter(circuit.setup_sweeps_names)) == "FS_Advanced Setup"
         variables = circuit.variable_manager.variables
-        assert variables["C1"].value == pytest.approx(1.967e-12)
-        assert variables["L2"].value == pytest.approx(1.288e-8)
-        assert variables["C3"].value == pytest.approx(6.366e-12)
-        app.desktop_class.close_desktop()
+        assert variables["C1"].si_value == pytest.approx(1.967e-12)
+        assert variables["L2"].si_value == pytest.approx(1.288e-8)
+        assert variables["C3"].si_value == pytest.approx(6.366e-12)
+        circuit.desktop_class.close_desktop()
 
     def test_distributed_circuit_exported_desktop(self, distributed_design):
         schem_name = distributed_design.export_to_aedt.schematic_name
         schem_name_length = len(schem_name)
         distributed_design.export_to_aedt.insert_circuit_design = True
-        app = distributed_design.export_to_aedt.export_design()
         circuit = distributed_design.export_to_aedt.export_design()
         assert isinstance(circuit, Circuit)
         assert circuit.project_name.split()[0][:schem_name_length] == schem_name
         assert circuit.design_name == schem_name
         assert next(iter(circuit.setup_sweeps_names)) == "FS_Advanced Setup"
         variables = circuit.variable_manager.variables
-        assert variables["W1"].value == pytest.approx(5.08e-3)
-        assert variables["W2"].value == pytest.approx(3.175e-4)
-        assert variables["W3"].value == pytest.approx(5.08e-3)
-        assert variables["S1"].value == pytest.approx(3.362e-3)
-        assert variables["S2"].value == pytest.approx(2.172e-2)
-        assert variables["S3"].value == pytest.approx(1.008e-2)
-        app.desktop_class.close_desktop()
+        assert variables["W1"].si_value == pytest.approx(5.08e-3)
+        assert variables["W2"].si_value == pytest.approx(3.175e-4)
+        assert variables["W3"].si_value == pytest.approx(5.08e-3)
+        assert variables["S1"].si_value == pytest.approx(3.362e-3)
+        assert variables["S2"].si_value == pytest.approx(2.172e-2)
+        assert variables["S3"].si_value == pytest.approx(1.008e-2)
+        circuit.desktop_class.close_desktop()
 
     def test_distributed_hfss3dl_exported_desktop(self, distributed_design):
         schem_name = distributed_design.export_to_aedt.schematic_name
         schem_name_length = len(schem_name)
         distributed_design.export_to_aedt.insert_hfss_3dl_design = True
-        app = distributed_design.export_to_aedt.export_design()
         hfss3dl = distributed_design.export_to_aedt.export_design()
         assert isinstance(hfss3dl, Hfss3dLayout)
         assert hfss3dl.project_name.split()[0][:schem_name_length] == schem_name
         assert hfss3dl.design_name == schem_name
         assert next(iter(hfss3dl.setup_sweeps_names)) == "FS_Advanced Setup"
         variables = hfss3dl.variable_manager.variables
-        assert variables["Win"].value == pytest.approx(1.234e-3)
-        assert variables["W1"].value == pytest.approx(5.08e-3)
-        assert variables["W2"].value == pytest.approx(3.175e-4)
-        assert variables["W3"].value == pytest.approx(5.08e-3)
-        assert variables["S1"].value == pytest.approx(3.36225452227e-3)
-        assert variables["S2"].value == pytest.approx(2.17231965814e-2)
-        assert variables["S3"].value == pytest.approx(1.00773795179e-2)
-        app.desktop_class.close_desktop()
+        assert variables["Win"].si_value == pytest.approx(1.234e-3)
+        assert variables["W1"].si_value == pytest.approx(5.08e-3)
+        assert variables["W2"].si_value == pytest.approx(3.175e-4)
+        assert variables["W3"].si_value == pytest.approx(5.08e-3)
+        assert variables["S1"].si_value == pytest.approx(3.36225452227e-3)
+        assert variables["S2"].si_value == pytest.approx(2.17231965814e-2)
+        assert variables["S3"].si_value == pytest.approx(1.00773795179e-2)
+        hfss3dl.desktop_class.close_desktop()
 
     def test_distributed_hfss_exported_desktop(self, distributed_design):
         schem_name = distributed_design.export_to_aedt.schematic_name
         schem_name_length = len(schem_name)
         distributed_design.export_to_aedt.insert_hfss_design = True
-        app = distributed_design.export_to_aedt.export_design()
         hfss = distributed_design.export_to_aedt.export_design()
         assert isinstance(hfss, Hfss)
         assert hfss.project_name.split()[0][:schem_name_length] == schem_name
         assert hfss.design_name == schem_name
         assert next(iter(hfss.setup_sweeps_names)) == "FS_Advanced_Setup"
         variables = hfss.variable_manager.variables
-        assert variables["Win"].value == pytest.approx(1.234e-3)
-        assert variables["W1"].value == pytest.approx(5.08e-3)
-        assert variables["W2"].value == pytest.approx(3.175e-4)
-        assert variables["W3"].value == pytest.approx(5.08e-3)
-        assert variables["S1"].value == pytest.approx(3.36225452227e-3)
-        assert variables["S2"].value == pytest.approx(2.17231965814e-2)
-        assert variables["S3"].value == pytest.approx(1.00773795179e-2)
-        app.desktop_class.close_desktop()
+        assert variables["Win"].si_value == pytest.approx(1.234e-3)
+        assert variables["W1"].si_value == pytest.approx(5.08e-3)
+        assert variables["W2"].si_value == pytest.approx(3.175e-4)
+        assert variables["W3"].si_value == pytest.approx(5.08e-3)
+        assert variables["S1"].si_value == pytest.approx(3.36225452227e-3)
+        assert variables["S2"].si_value == pytest.approx(2.17231965814e-2)
+        assert variables["S3"].si_value == pytest.approx(1.00773795179e-2)
+        hfss.desktop_class.close_desktop()

tests/system/filter_solutions/test_export_to_aedt/test_export_to_aedt.py

@@ -35,7 +35,6 @@
 from ansys.aedt.core.generic.settings import is_linux
 from tests.conftest import DESKTOP_VERSION
 from tests.conftest import SKIP_MODELITHICS
-from tests.conftest import USE_GRPC
 from tests.system.filter_solutions.resources import read_resource_file
 from tests.system.filter_solutions.resources import resource_path
 
@@ -472,9 +471,7 @@ def test_save_library_parts_config(self, lumped_design):
         assert lumped_design.export_to_aedt.substrate_dielectric_height == "3 mm"
         assert lumped_design.export_to_aedt.substrate_loss_tangent == "0.065 "
 
-    @pytest.mark.skipif(
-        DESKTOP_VERSION < "2026.1" or USE_GRPC, reason="Skipped on versions earlier than 2026.1 or gRPC mode"
-    )
+    @pytest.mark.skipif(DESKTOP_VERSION < "2026.1", reason="Skipped on versions earlier than 2026.1")
     def test_import_tuned_variables(self, lumped_design):
         lumped_design.export_to_aedt.simulate_after_export_enabled = True
         lumped_design.export_to_aedt.optimize_after_export_enabled = True