From 39584ffa3c388a1bd7d8344895197abbc4666266 Mon Sep 17 00:00:00 2001 From: Prithwish Mukherjee Date: Fri, 28 Nov 2025 16:29:47 +0530 Subject: [PATCH 1/6] fix: Pop up window for launch fluent. --- src/ansys/fluent/core/launcher/launcher_utils.py | 7 +++++-- src/ansys/fluent/core/launcher/standalone_launcher.py | 6 +++++- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/src/ansys/fluent/core/launcher/launcher_utils.py b/src/ansys/fluent/core/launcher/launcher_utils.py index 4bcf0b19921b..fca18630af05 100644 --- a/src/ansys/fluent/core/launcher/launcher_utils.py +++ b/src/ansys/fluent/core/launcher/launcher_utils.py @@ -106,9 +106,12 @@ def _get_subprocess_kwargs_for_fluent(env: Dict[str, Any], argvals) -> Dict[str, stderr=pyfluent.config.launch_fluent_stderr, ) if is_windows(): - kwargs.update(shell=True, creationflags=subprocess.CREATE_NEW_PROCESS_GROUP) + kwargs.update( + creationflags=subprocess.CREATE_NEW_PROCESS_GROUP + | subprocess.CREATE_NO_WINDOW + ) else: - kwargs.update(shell=True, start_new_session=True) + kwargs.update(start_new_session=True) fluent_env = os.environ.copy() fluent_env.update({k: str(v) for k, v in env.items()}) fluent_env["REMOTING_THROW_LAST_TUI_ERROR"] = "1" diff --git a/src/ansys/fluent/core/launcher/standalone_launcher.py b/src/ansys/fluent/core/launcher/standalone_launcher.py index e30128bfd320..34004731fffa 100644 --- a/src/ansys/fluent/core/launcher/standalone_launcher.py +++ b/src/ansys/fluent/core/launcher/standalone_launcher.py @@ -257,7 +257,11 @@ def __call__(self): return self._launch_string, self._server_info_file_name try: logger.debug(f"Launching Fluent with command: {self._launch_cmd}") - + if isinstance(self._launch_cmd, str) and self._launch_cmd.startswith( + "start " + ): + parts = self._launch_cmd.split() + self._launch_cmd = parts[2:] # Remove: start "" process = subprocess.Popen(self._launch_cmd, **self._kwargs) try: From e50396b5c5bf8a83e19974e3ffb8f115f609f0de Mon Sep 17 00:00:00 2001 From: pyansys-ci-bot <92810346+pyansys-ci-bot@users.noreply.github.com> Date: Fri, 28 Nov 2025 11:03:43 +0000 Subject: [PATCH 2/6] chore: adding changelog file 4670.fixed.md [dependabot-skip] --- doc/changelog.d/4670.fixed.md | 1 + 1 file changed, 1 insertion(+) create mode 100644 doc/changelog.d/4670.fixed.md diff --git a/doc/changelog.d/4670.fixed.md b/doc/changelog.d/4670.fixed.md new file mode 100644 index 000000000000..4e58294182a1 --- /dev/null +++ b/doc/changelog.d/4670.fixed.md @@ -0,0 +1 @@ +Pop up window for launch fluent. From e452bbe0e2afdf4b56fac6b5ece315aef78c6a69 Mon Sep 17 00:00:00 2001 From: Prithwish Mukherjee Date: Fri, 28 Nov 2025 16:50:43 +0530 Subject: [PATCH 3/6] Update. --- src/ansys/fluent/core/launcher/standalone_launcher.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/ansys/fluent/core/launcher/standalone_launcher.py b/src/ansys/fluent/core/launcher/standalone_launcher.py index 34004731fffa..9f2bdd111ba8 100644 --- a/src/ansys/fluent/core/launcher/standalone_launcher.py +++ b/src/ansys/fluent/core/launcher/standalone_launcher.py @@ -261,7 +261,9 @@ def __call__(self): "start " ): parts = self._launch_cmd.split() - self._launch_cmd = parts[2:] # Remove: start "" + self._launch_cmd = ( + parts[2:] if len(parts) > 2 else parts[:] + ) # Remove: start "" process = subprocess.Popen(self._launch_cmd, **self._kwargs) try: From aadd246c7026e00413749e2224f6355505d453a0 Mon Sep 17 00:00:00 2001 From: Prithwish Mukherjee <109645853+prmukherj@users.noreply.github.com> Date: Fri, 28 Nov 2025 16:54:43 +0530 Subject: [PATCH 4/6] Update doc/changelog.d/4670.fixed.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- doc/changelog.d/4670.fixed.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/changelog.d/4670.fixed.md b/doc/changelog.d/4670.fixed.md index 4e58294182a1..e64a6bf7b813 100644 --- a/doc/changelog.d/4670.fixed.md +++ b/doc/changelog.d/4670.fixed.md @@ -1 +1 @@ -Pop up window for launch fluent. +Fixed popup window appearing when launching Fluent in no_graphics or no_gui_or_graphics modes. From 4b85d7f1eae44b47cf7b63eb2e223a7f7cc527aa Mon Sep 17 00:00:00 2001 From: pyansys-ci-bot <92810346+pyansys-ci-bot@users.noreply.github.com> Date: Fri, 28 Nov 2025 11:25:34 +0000 Subject: [PATCH 5/6] chore: adding changelog file 4670.fixed.md [dependabot-skip] --- doc/changelog.d/4670.fixed.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/changelog.d/4670.fixed.md b/doc/changelog.d/4670.fixed.md index e64a6bf7b813..4e58294182a1 100644 --- a/doc/changelog.d/4670.fixed.md +++ b/doc/changelog.d/4670.fixed.md @@ -1 +1 @@ -Fixed popup window appearing when launching Fluent in no_graphics or no_gui_or_graphics modes. +Pop up window for launch fluent. From 81ed70a4689a78c2e3909cfc717161d2dbe67dbb Mon Sep 17 00:00:00 2001 From: Prithwish Mukherjee Date: Fri, 28 Nov 2025 20:31:37 +0530 Subject: [PATCH 6/6] Fix in initialization. --- .../fluent/core/launcher/standalone_launcher.py | 16 +--------------- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/src/ansys/fluent/core/launcher/standalone_launcher.py b/src/ansys/fluent/core/launcher/standalone_launcher.py index 9f2bdd111ba8..fc06ae0da792 100644 --- a/src/ansys/fluent/core/launcher/standalone_launcher.py +++ b/src/ansys/fluent/core/launcher/standalone_launcher.py @@ -236,14 +236,7 @@ def __init__( ) if is_windows(): - if ( - pyfluent.config.launch_fluent_stdout - or pyfluent.config.launch_fluent_stderr - ): - self._launch_cmd = self._launch_string - else: - # Using 'start.exe' is better; otherwise Fluent is more susceptible to bad termination attempts. - self._launch_cmd = 'start "" ' + self._launch_string + self._launch_cmd = self._launch_string else: if self.argvals["ui_mode"] not in [UIMode.GUI, UIMode.HIDDEN_GUI]: # Using nohup to hide Fluent output from the current terminal @@ -257,13 +250,6 @@ def __call__(self): return self._launch_string, self._server_info_file_name try: logger.debug(f"Launching Fluent with command: {self._launch_cmd}") - if isinstance(self._launch_cmd, str) and self._launch_cmd.startswith( - "start " - ): - parts = self._launch_cmd.split() - self._launch_cmd = ( - parts[2:] if len(parts) > 2 else parts[:] - ) # Remove: start "" process = subprocess.Popen(self._launch_cmd, **self._kwargs) try: