build: Bump version 0.30.1 (#3831)

Co-authored-by: pyansys-ci-bot <[email protected]>

Author: mkundu1

doc/changelog.d/3832.added.md

@@ -0,0 +1 @@
+Remove lxml

pyproject.toml

@@ -27,7 +27,6 @@ dependencies = [
     "grpcio>=1.30.0",
     "grpcio-health-checking>=1.30.0",
     "grpcio-status>=1.26.0",
-    "lxml>=4.9.2",
     "nltk>=3.9.1",
     "numpy>=1.14.0,<3.0.0",
     "pandas>=1.1.0,<2.3",

src/ansys/fluent/core/__init__.py

@@ -62,7 +62,7 @@
 from ansys.fluent.core.utils.fluent_version import FluentVersion  # noqa: F401
 from ansys.fluent.core.utils.setup_for_fluent import setup_for_fluent  # noqa: F401
 
-__version__ = "0.30.0"
+__version__ = "0.30.1"
 
 _VERSION_INFO = None
 """

src/ansys/fluent/core/codegen/settingsgen.py

@@ -226,11 +226,11 @@ def _write_data(cls_name: str, python_name: str, data: dict, f: IO, f_stub: IO |
     s.write('    """\n')
     s.write(f"    {doc}\n")
     s.write('    """\n')
-    s.write(f"    version = {data['version']!r}\n")
+    s.write(f"    _version = {data['version']!r}\n")
     s.write(f"    fluent_name = {data['fluent_name']!r}\n")
     # _python_name preserves the original non-suffixed name of the class.
     s.write(f"    _python_name = {python_name!r}\n")
-    s_stub.write("    version: str\n")
+    s_stub.write("    _version: str\n")
     s_stub.write("    fluent_name: str\n")
     s_stub.write("    _python_name: str\n")
     child_names = data["child_names"]

src/ansys/fluent/core/filereader/case_file.py

@@ -46,7 +46,6 @@
 from typing import Dict, List
 import xml.etree.ElementTree as ET
 
-from lxml import etree
 import numpy as np
 
 from ansys.fluent.core.solver.error_message import allowed_name_error_message
@@ -732,8 +731,7 @@ def _get_processed_string(input_string: bytes) -> str:
 
 
 def _get_case_file_name_from_flprj(flprj_file):
-    parser = etree.XMLParser(recover=True)
-    tree = ET.parse(flprj_file, parser)
+    tree = ET.parse(flprj_file)
     root = tree.getroot()
     folder_name = root.find("Metadata").find("CurrentSimulation").get("value")[5:-1]
     # If the project file name begins with a digit then the node to find will be prepended

src/ansys/fluent/core/filereader/data_file.py

@@ -40,7 +40,6 @@
 from pathlib import Path
 import xml.etree.ElementTree as ET
 
-from lxml import etree
 import numpy as np
 
 from . import lispy
@@ -225,8 +224,7 @@ def get_face_vector_field_data(self, phase_name: str, surface_id: int) -> np.arr
 
 
 def _get_data_file_name_from_flprj(flprj_file):
-    parser = etree.XMLParser(recover=True)
-    tree = ET.parse(flprj_file, parser)
+    tree = ET.parse(flprj_file)
     root = tree.getroot()
     folder_name = root.find("Metadata").find("CurrentSimulation").get("value")[5:-1]
     return root.find(folder_name).find("Input").find("Case").find("Target").get("value")

src/ansys/fluent/core/report.py

@@ -64,7 +64,6 @@
     "grpcio-health-checking": "1.30.0",
     "grpcio-status": "1.30.0",
     "h5py": "3.12.1",
-    "lxml": "4.9.2",
     "nltk": "3.9.1",
     "numpy": "1.14.0,<3.0.0",
     "pandas": "1.1.0,<2.3",

src/ansys/fluent/core/solver/flobject.py

@@ -347,7 +347,7 @@ def python_path(self) -> str:
         Constructed in python syntax from 'python_path' and the parents python path.
         """
         if self._parent is None:
-            if FluentVersion(self.version).number < 251:
+            if FluentVersion(self._version).number < 251:
                 return "<session>"
             else:
                 return "<session>.settings"
@@ -413,7 +413,7 @@ def _is_deprecated(self) -> bool:
         deprecated_version = (
             deprecated_version.get("deprecated-version") if deprecated_version else None
         )
-        return deprecated_version and FluentVersion(self.version) >= FluentVersion(
+        return deprecated_version and FluentVersion(self._version) >= FluentVersion(
             deprecated_version
         )
 
@@ -2035,7 +2035,7 @@ def get_cls(name, info, parent=None, version=None, parent_taboo=None):
                 f"Falling back to String."
             )
             base = String
-        dct = {"fluent_name": name, "version": version}
+        dct = {"fluent_name": name, "_version": version}
         helpinfo = info.get("help")
         if helpinfo:
             dct["__doc__"] = _fix_help_info(obj_type, _clean_helpinfo(helpinfo))

src/ansys/fluent/core/solver/settings_builtin_bases.py

@@ -55,7 +55,7 @@ def _get_settings_obj(settings_root, builtin_settings_obj):
     obj = settings_root
     path = DATA[builtin_cls_name][1]
     if isinstance(path, dict):
-        version = FluentVersion(obj.version)
+        version = FluentVersion(obj._version)
         path = path.get(version)
         if path is None:
             raise RuntimeError(

tests/test_codegen.py

@@ -480,15 +480,15 @@ class P3(Integer):
     """
     P3 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'P3'
     _python_name = 'P3'
 
 class G2(Group):
     """
     G2 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'G2'
     _python_name = 'G2'
     child_names = ['P3']
@@ -500,15 +500,15 @@ class P2(Real):
     """
     P2 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'P2'
     _python_name = 'P2'
 
 class A2(Real):
     """
     A2 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'A2'
     _python_name = 'A2'
 
@@ -521,7 +521,7 @@ class C2(Command):
         A2 : real
             A2 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'C2'
     _python_name = 'C2'
     argument_names = ['A2']
@@ -538,7 +538,7 @@ class Q2(Query):
         A2 : real
             A2 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'Q2'
     _python_name = 'Q2'
     argument_names = ['A2']
@@ -550,7 +550,7 @@ class G1(Group):
     """
     G1 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'G1'
     _python_name = 'G1'
     child_names = ['G2', 'P2']
@@ -567,31 +567,31 @@ class P1(String):
     """
     P1 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'P1'
     _python_name = 'P1'
 
 class P4(String):
     """
     P4 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'P4'
     _python_name = 'P4'
 
 class N1_child(Group):
     """
     'child_object_type' of N1.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'child-object-type'
     _python_name = 'N1_child'
 
 class N1(NamedObject[N1_child], _NonCreatableNamedObjectMixin[N1_child]):
     """
     N1 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'N1'
     _python_name = 'N1'
     child_names = ['P4']
@@ -604,7 +604,7 @@ class A1(String):
     """
     A1 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'A1'
     _python_name = 'A1'
 
@@ -617,7 +617,7 @@ class C1(Command):
         A1 : str
             A1 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'C1'
     _python_name = 'C1'
     argument_names = ['A1']
@@ -634,7 +634,7 @@ class Q1(Query):
         A1 : str
             A1 help.
     """
-    version = '251'
+    _version = '251'
     fluent_name = 'Q1'
     _python_name = 'Q1'
     argument_names = ['A1']
@@ -646,7 +646,7 @@ class root(Group):
     """
     'root' object.
     """
-    version = '251'
+    _version = '251'
     fluent_name = ''
     _python_name = 'root'
     child_names = ['G1', 'P1', 'N1']

tests/test_flobject.py

@@ -538,7 +538,7 @@ def test_command():
 
 def test_attrs():
     r = flobject.get_root(Proxy())
-    r._setattr("version", "251")
+    r._setattr("_version", "251")
     assert r.g_1.s_4.get_attr("active?")
     assert r.g_1.s_4.get_attr("allowed-values") == ["foo", "bar"]
     r.g_1.b_3 = True

tests/test_type_stub.py

@@ -46,7 +46,7 @@ def test_settings_stub():
     assert len(class_def.bases) > 0
     assigns = [x for x in class_def.body if isinstance(x, ast.AnnAssign)]
     for class_attr in [
-        "version",
+        "_version",
         "fluent_name",
         "_python_name",
         "child_names",