chore: cleanup deprecated methods (#1860)

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

Author: RobPasMue

doc/changelog.d/1860.maintenance.md

@@ -0,0 +1 @@
+cleanup deprecated methods

doc/source/assets.rst

@@ -69,7 +69,7 @@ Windows container
    Only users with access to https://github.com/ansys/pyansys-geometry-binaries can download these binaries.
 
 * `Latest Geometry service binaries for Windows containers <https://github.com/ansys/pyansys-geometry-binaries>`_
-* `Latest Geometry service Dockerfile for Windows containers <https://github.com/ansys/pyansys-geometry/releases/latest/download/windows-dms-dockerfile.zip>`_
+* `Latest Geometry service Dockerfile for Windows containers <https://github.com/ansys/pyansys-geometry/releases/latest/download/windows-core-dockerfile.zip>`_
 
 .. Linux container
 .. ^^^^^^^^^^^^^^^

src/ansys/geometry/core/connection/launcher.py

@@ -35,7 +35,7 @@
 )
 from ansys.geometry.core.connection.product_instance import prepare_and_start_backend
 from ansys.geometry.core.logger import LOG
-from ansys.geometry.core.misc.checks import check_type, deprecated_argument
+from ansys.geometry.core.misc.checks import check_type
 
 try:
     import ansys.platform.instancemanagement as pypim
@@ -492,12 +492,6 @@ def launch_modeler_with_spaceclaim_and_pimlight(
     )
 
 
-@deprecated_argument(
-    arg="log_level", alternative="server_log_level", version="0.6.2", remove="0.10.0"
-)
-@deprecated_argument(
-    arg="logs_folder", alternative="server_logs_folder", version="0.6.2", remove="0.10.0"
-)
 def launch_modeler_with_geometry_service(
     product_version: int = None,
     host: str = "localhost",
@@ -508,8 +502,6 @@ def launch_modeler_with_geometry_service(
     client_log_level: int = logging.INFO,
     server_logs_folder: str = None,
     client_log_file: str = None,
-    log_level: int = None,  # DEPRECATED
-    logs_folder: str = None,  # DEPRECATED
     **kwargs: dict | None,
 ) -> "Modeler":
     """Start the Geometry service locally using the ``ProductInstance`` class.
@@ -559,10 +551,6 @@ def launch_modeler_with_geometry_service(
     client_log_file : str, optional
         Sets the client's log file path. If nothing is defined,
         the client will log to the console.
-    log_level : int, optional
-        DEPRECATED. Use ``server_log_level`` instead.
-    logs_folder : str, optional
-        DEPRECATED. Use ``server_logs_folder`` instead.
     **kwargs : dict, default: None
         Placeholder to prevent errors when passing additional arguments that
         are not compatible with this method.
@@ -625,14 +613,9 @@ def launch_modeler_with_geometry_service(
         client_log_level=client_log_level,
         server_logs_folder=server_logs_folder,
         client_log_file=client_log_file,
-        log_level=log_level,
-        logs_folder=logs_folder,
     )
 
 
-@deprecated_argument(
-    arg="log_level", alternative="server_log_level", version="0.6.2", remove="0.10.0"
-)
 def launch_modeler_with_discovery(
     product_version: int = None,
     host: str = "localhost",
@@ -644,7 +627,6 @@ def launch_modeler_with_discovery(
     server_log_level: int = 2,
     client_log_level: int = logging.INFO,
     client_log_file: str = None,
-    log_level: int = None,  # DEPRECATED
     **kwargs: dict | None,
 ):
     """Start Ansys Discovery locally using the ``ProductInstance`` class.
@@ -696,8 +678,6 @@ def launch_modeler_with_discovery(
     client_log_file : str, optional
         Sets the client's log file path. If nothing is defined,
         the client will log to the console.
-    log_level : int, optional
-        DEPRECATED. Use ``server_log_level`` instead.
     **kwargs : dict, default: None
         Placeholder to prevent errors when passing additional arguments that
         are not compatible with this method.
@@ -755,13 +735,9 @@ def launch_modeler_with_discovery(
         server_log_level=server_log_level,
         client_log_level=client_log_level,
         client_log_file=client_log_file,
-        log_level=log_level,
     )
 
 
-@deprecated_argument(
-    arg="log_level", alternative="server_log_level", version="0.6.2", remove="0.10.0"
-)
 def launch_modeler_with_spaceclaim(
     product_version: int = None,
     host: str = "localhost",
@@ -773,7 +749,6 @@ def launch_modeler_with_spaceclaim(
     server_log_level: int = 2,
     client_log_level: int = logging.INFO,
     client_log_file: str = None,
-    log_level: int = None,  # DEPRECATED
     **kwargs: dict | None,
 ):
     """Start Ansys SpaceClaim locally using the ``ProductInstance`` class.
@@ -825,8 +800,6 @@ def launch_modeler_with_spaceclaim(
     client_log_file : str, optional
         Sets the client's log file path. If nothing is defined,
         the client will log to the console.
-    log_level : int, optional
-        DEPRECATED. Use ``server_log_level`` instead.
     **kwargs : dict, default: None
         Placeholder to prevent errors when passing additional arguments that
         are not compatible with this method.
@@ -884,7 +857,6 @@ def launch_modeler_with_spaceclaim(
         server_log_level=server_log_level,
         client_log_level=client_log_level,
         client_log_file=client_log_file,
-        log_level=log_level,
     )
 
 

src/ansys/geometry/core/connection/product_instance.py

@@ -189,8 +189,6 @@ def prepare_and_start_backend(
     server_logs_folder: str = None,
     client_log_file: str = None,
     specific_minimum_version: int = None,
-    log_level: int = None,  # DEPRECATED
-    logs_folder: str = None,  # DEPRECATED
 ) -> "Modeler":
     """Start the requested service locally using the ``ProductInstance`` class.
 
@@ -245,10 +243,6 @@ def prepare_and_start_backend(
     specific_minimum_version : int, optional
         Sets a specific minimum version to be checked. If this is not defined,
         the minimum version will be set to 24.1.0.
-    log_level : int, optional
-        DEPRECATED. Use ``server_log_level`` instead.
-    logs_folder : str, optional
-        DEPRECATED. Use ``server_logs_folder`` instead.
 
     Returns
     -------
@@ -266,24 +260,12 @@ def prepare_and_start_backend(
     """
     from ansys.geometry.core.modeler import Modeler
 
-    if os.name != "nt" and backend_type not in (
-        BackendType.LINUX_SERVICE,
-        BackendType.CORE_LINUX,
-    ):  # pragma: no cover
+    if os.name != "nt" and not BackendType.is_linux_service(backend_type):  # pragma: no cover
         raise RuntimeError(
             "Method 'prepare_and_start_backend' is only available on Windows."
             "A Linux version is only available for the Core Geometry Service."
         )
 
-    # Deprecation behavior... To be removed in release 0.7
-    if log_level is not None:  # pragma: no cover
-        LOG.warning("Overriding 'server_log_level' with 'log_level' value for now...")
-        server_log_level = log_level
-
-    if logs_folder is not None:  # pragma: no cover
-        LOG.warning("Overriding 'server_logs_folder' with 'logs_folder' value for now...")
-        server_logs_folder = logs_folder
-
     port = _check_port_or_get_one(port)
     installations = get_available_ansys_installations()
     if os.getenv(ANSYS_GEOMETRY_SERVICE_ROOT) is not None and backend_type in (

src/ansys/geometry/core/designer/face.py

@@ -57,7 +57,6 @@
     convert_opacity_to_hex,
 )
 from ansys.geometry.core.misc.checks import (
-    deprecated_method,
     ensure_design_is_active,
     graphics_required,
     min_backend_version,
@@ -414,32 +413,6 @@ def normal(self, u: float = 0.5, v: float = 0.5) -> UnitVector3D:
             response = self._faces_stub.GetNormal(GetNormalRequest(id=self.id, u=u, v=v)).direction
             return UnitVector3D([response.x, response.y, response.z])
 
-    @deprecated_method(alternative="normal", version="0.6.2", remove="0.10.0")
-    def face_normal(self, u: float = 0.5, v: float = 0.5) -> UnitVector3D:
-        """Get the normal direction to the face at certain UV coordinates.
-
-        Parameters
-        ----------
-        u : float, default: 0.5
-            First coordinate of the 2D representation of a surface in UV space.
-            The default is ``0.5``, which is the center of the surface.
-        v : float, default: 0.5
-            Second coordinate of the 2D representation of a surface in UV space.
-            The default is ``0.5``, which is the center of the surface.
-
-        Returns
-        -------
-        UnitVector3D
-            :class:`UnitVector3D` object evaluated at the given U and V coordinates.
-            This :class:`UnitVector3D` object is perpendicular to the surface at the
-            given UV coordinates.
-
-        Notes
-        -----
-        This method is deprecated. Use the ``normal`` method instead.
-        """
-        return self.normal(u, v)
-
     @protect_grpc
     @ensure_design_is_active
     def point(self, u: float = 0.5, v: float = 0.5) -> Point3D:
@@ -473,30 +446,6 @@ def point(self, u: float = 0.5, v: float = 0.5) -> Point3D:
             response = self._faces_stub.Evaluate(EvaluateRequest(id=self.id, u=u, v=v)).point
             return Point3D([response.x, response.y, response.z], DEFAULT_UNITS.SERVER_LENGTH)
 
-    @deprecated_method(alternative="point", version="0.6.2", remove="0.10.0")
-    def face_point(self, u: float = 0.5, v: float = 0.5) -> Point3D:
-        """Get a point of the face evaluated at certain UV coordinates.
-
-        Parameters
-        ----------
-        u : float, default: 0.5
-            First coordinate of the 2D representation of a surface in UV space.
-            The default is ``0.5``, which is the center of the surface.
-        v : float, default: 0.5
-            Second coordinate of the 2D representation of a surface in UV space.
-            The default is ``0.5``, which is the center of the surface.
-
-        Returns
-        -------
-        Point3D
-            :class:`Point3D` object evaluated at the given UV coordinates.
-
-        Notes
-        -----
-        This method is deprecated. Use the ``point`` method instead.
-        """
-        return self.point(u, v)
-
     def __grpc_edges_to_edges(self, edges_grpc: list[GRPCEdge]) -> list[Edge]:
         """Transform a list of gRPC edge messages into actual ``Edge`` objects.