fix(migrated PR 4207): issue 4206 (#4279)

* Initial fix in interp_star_status function

* Removed junk lines and fixed a typo

* ci: auto fixes from pre-commit.com hooks.

for more information, see https://pre-commit.ci

* chore: adding changelog file 4279.fixed.md [dependabot-skip]

* chore: adding changelog file 4279.fixed.md [dependabot-skip]

* style: add type annotations to parameter parsing helpers in parameters.py

Add explicit type hints for math_header, array_header, is_parameter_listing,
is_math_listing, is_array_listing, and interp_star_status to improve clarity
and support static analysis.

* test: add tests for C_FullFile parameter handling in interp_star_status

Add golden "c_fullfile status" sample and multiple tests to verify
interp_star_status correctly parses C_FullFile entries. Cover mixed
SMAT/C_FullFile outputs, single C_FullFile outputs, and various
C_FullFile-like names; ensure only the "type" field is present for
C_FullFile parameters and other fields (value, MemoryMB, workspace,
shape) are not included. Also include the c_fullfile case in the
parameterized status test.

---------

Co-authored-by: Chai Fillerup <[email protected]>
Co-authored-by: ChaiF1 <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Camille Latapie <[email protected]>
Co-authored-by: German <[email protected]>

Author: 5 people

doc/changelog.d/4279.fixed.md

@@ -0,0 +1 @@
+Issue 4206

src/ansys/mapdl/core/parameters.py

@@ -722,27 +722,27 @@ def parameter_header(line):
     return "NAME" in line and "VALUE" in line and "TYPE" in line
 
 
-def math_header(line):
+def math_header(line: str) -> bool:
     return "Name" in line and "Type" in line and "Dims" in line and "Workspace" in line
 
 
-def array_header(line):
+def array_header(line: str) -> bool:
     return "LOCATION" in line and "VALUE" in line
 
 
-def is_parameter_listing(status):
+def is_parameter_listing(status: str) -> bool:
     return any([parameter_header(each) for each in status.splitlines()])
 
 
-def is_math_listing(status):
+def is_math_listing(status: str) -> bool:
     return any([math_header(each) for each in status.splitlines()])
 
 
-def is_array_listing(status):
+def is_array_listing(status: str) -> bool:
     return any([array_header(each) for each in status.splitlines()])
 
 
-def interp_star_status(status):
+def interp_star_status(status: str) -> dict[str, str]:
     """Interprets \\*STATUS command output from MAPDL
 
     Parameters
@@ -853,6 +853,8 @@ def interp_star_status(status):
                     "type": items[1],
                     "workspace": int(items[4]),
                 }
+            elif items[1] in ["C_FullFile"]:
+                parameters[name] = {"type": items[1]}
             else:
                 shape = (int(items[2]), int(items[3]), int(items[4]))
                 parameters[name] = {"type": items[1], "shape": shape}

tests/test_parameters.py

@@ -104,6 +104,13 @@
  PGFZJK_ROWDIM                     20.0000000                  SCALAR
  PORT                              50054.0000                  SCALAR
  STRARRAY                                   STRING ARRAY      96       1       1""",
+    "c_fullfile status": """APDLMATH PARAMETER STATUS-  (      3 PARAMETERS DEFINED)
+
+  Name                   Type            Mem. (MB)       Dims            Workspace
+
+   K                     SMAT            4.720           [19764:19764]           1
+   M                     SMAT            2.728           [19764:19764]           1
+   file.full             C_FullFile                      --              --              --""",
 }
 
 
@@ -355,6 +362,7 @@ def test_parameter_delete_raise(mapdl, cleared):
         ),
         ("string array status", ["aqzzzxcv zx zxcv   zxcv", "qwer wer qwer", "zxcv"]),
         ("general status", None),
+        ("c_fullfile status", None),
     ],
 )
 def test_interp_star_status(status_key, check):
@@ -364,8 +372,15 @@ def test_interp_star_status(status_key, check):
         name = list(output.keys())[0]
         if name == "MYARR" or name == "ASDF":
             assert np.all(output[name]["value"] == check)
+        elif name == "file.full":
+            assert output[name]["type"] == "C_FullFile"
+            assert "value" not in output[name]
         else:
             assert output[name]["value"] == check
+    elif status_key == "c_fullfile status":
+        assert "file.full" in output
+        assert output["file.full"]["type"] == "C_FullFile"
+        assert "value" not in output["file.full"]
     else:
         assert "MYARR" in output
         assert "PGFZJK_COLDIM" in output
@@ -534,6 +549,91 @@ def test_parameter_types(mapdl, cleared, parameter):
         assert isinstance(mapdl.parameters["temp_arr"], type(parameter))
 
 
+def test_c_fullfile_parameter_handling_mixed():
+    """Test handling of C_FullFile parameter type in interp_star_status function.
+
+    This test covers issue #4206 where imported matrix/vector files from .full files
+    are reported as C_FullFile parameters and need special handling.
+    """
+    # Test status output representing a mix of SMAT and C_FullFile parameters
+    status_mixed = """APDLMATH PARAMETER STATUS-  (      3 PARAMETERS DEFINED)
+
+  Name                   Type            Mem. (MB)       Dims            Workspace
+
+   K                     SMAT            4.720           [19764:19764]           1
+   M                     SMAT            2.728           [19764:19764]           1
+   file.full             C_FullFile                      --              --              --"""
+
+    # Test mixed status
+    output_mixed = interp_star_status(status_mixed)
+
+    # Should have all 3 parameters
+    assert len(output_mixed) == 3
+
+    # Check SMAT parameters
+    assert "K" in output_mixed
+    assert output_mixed["K"]["type"] == "SMAT"
+    assert output_mixed["K"]["MemoryMB"] == 4.720
+    assert output_mixed["K"]["workspace"] == 1
+
+    assert "M" in output_mixed
+    assert output_mixed["M"]["type"] == "SMAT"
+    assert output_mixed["M"]["MemoryMB"] == 2.728
+    assert output_mixed["M"]["workspace"] == 1
+
+    # Check C_FullFile parameter - should only have type, no other fields
+    assert "file.full" in output_mixed
+    assert output_mixed["file.full"]["type"] == "C_FullFile"
+    assert "MemoryMB" not in output_mixed["file.full"]
+    assert "workspace" not in output_mixed["file.full"]
+    assert "value" not in output_mixed["file.full"]
+    assert "shape" not in output_mixed["file.full"]
+
+
+def test_c_fullfile_parameter_handling_single():
+    # Test single C_FullFile status
+
+    # Test status output with only C_FullFile parameter
+    status_single = """APDLMATH PARAMETER STATUS-  (      1 PARAMETERS DEFINED)
+
+  Name                   Type            Mem. (MB)       Dims            Workspace
+
+   myfile.full           C_FullFile                      --              --              --"""
+
+    output_single = interp_star_status(status_single)
+
+    assert len(output_single) == 1
+    assert "myfile.full" in output_single
+    assert output_single["myfile.full"]["type"] == "C_FullFile"
+    assert len(output_single["myfile.full"]) == 1  # Only 'type' field should be present
+
+
+@pytest.mark.parametrize(
+    "param_name,param_type",
+    [
+        ("matrix.full", "C_FullFile"),
+        ("stiffness.full", "C_FullFile"),
+        ("mass.full", "C_FullFile"),
+        ("mydata.full", "C_FullFile"),
+    ],
+)
+def test_c_fullfile_parameter_types(param_name, param_type):
+    """Test various C_FullFile parameter names and ensure proper handling."""
+    status = f"""APDLMATH PARAMETER STATUS-  (      1 PARAMETERS DEFINED)
+
+  Name                   Type            Mem. (MB)       Dims            Workspace
+
+   {param_name:<20}  {param_type:<15}                --              --              --"""
+
+    output = interp_star_status(status)
+
+    assert len(output) == 1
+    assert param_name in output
+    assert output[param_name]["type"] == param_type
+    # Ensure no other fields are present for C_FullFile parameters
+    assert len(output[param_name]) == 1
+
+
 @pytest.mark.parametrize("use_load_table", [True, False])
 def test_table_interpolation(mapdl, use_load_table):
     file_name = "table.txt"