Commit 5b5187e
chore: Update launcher tests for better coverage (#4629)
## Context
Extensive test coverage for the launcher APIs from session classes is
introduced.
One issue is when a 'PureMeshing' session is launched from PyFluent and
'Meshing' session is connected to it using the launcher api, it might be
confusing for users, as now no mechanism exists in server to inform
client that it has been launched in 'PureMeshing' mode. **Hence, this
limitation is to be documented.**
## Change Summary
Tests were added for extensive coverage of launcher APIs and the
limitation is documented.
## Impact
No direct impact, only a limitation as mentioned above to be documented.
**There are four possible connection patterns:**
Meshing → Meshing | ✔ Safe | Full capabilities preserved
PureMeshing → PureMeshing | ✔ Safe | Capabilities match exactly
Meshing → PureMeshing | ✔ Allowed | API connects to a subset of
available capabilities
PureMeshing → Meshing | ⚠ Not recommended | Meshing API expects solver
features that a pure-meshing session cannot provide
The fourth scenario is problematic because Meshing may assume
capabilities (e.g., solver mode switching) that are not enabled in a
pure-meshing session, leading to inconsistent or undefined behavior.
In later Fluent release we might consider updating the app-utilities, as
suggested by @mkundu1
"An implementation can be done by adding a new method
set_pure_meshing_app_mode in the app_utilities service. The new method
will be called to set a flag in Fluent when the Fluent session is
launched or connected from a PureMeshing PyFluent session. That flag
will be checked in get_app_modemethod of app_utilities. We can also
block switch-to-solver operation within Fluent when that flag is set."
---------
Co-authored-by: pyansys-ci-bot <[email protected]>1 parent 7b0e8d1 commit 5b5187e
File tree
4 files changed
+60
-6
lines changed- doc
- changelog.d
- source/user_guide/session
- src/ansys/fluent/core
- tests
4 files changed
+60
-6
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
100 | 100 | | |
101 | 101 | | |
102 | 102 | | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
103 | 123 | | |
104 | 124 | | |
105 | 125 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
41 | 41 | | |
42 | 42 | | |
43 | 43 | | |
44 | | - | |
| 44 | + | |
45 | 45 | | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
51 | 54 | | |
52 | 55 | | |
53 | 56 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
695 | 695 | | |
696 | 696 | | |
697 | 697 | | |
| 698 | + | |
| 699 | + | |
| 700 | + | |
| 701 | + | |
| 702 | + | |
| 703 | + | |
| 704 | + | |
| 705 | + | |
| 706 | + | |
| 707 | + | |
| 708 | + | |
| 709 | + | |
| 710 | + | |
| 711 | + | |
| 712 | + | |
| 713 | + | |
| 714 | + | |
| 715 | + | |
| 716 | + | |
| 717 | + | |
| 718 | + | |
| 719 | + | |
| 720 | + | |
| 721 | + | |
| 722 | + | |
| 723 | + | |
| 724 | + | |
| 725 | + | |
| 726 | + | |
| 727 | + | |
698 | 728 | | |
699 | 729 | | |
700 | 730 | | |
| |||
0 commit comments