[FXC-2006] Disable boundary completeness and deletion validation (#1330)

* [FSC-2006] Disable boundary completness and deletion validation

* Fix unit test

Author: benflexcompute

flow360/component/simulation/primitives.py

@@ -514,6 +514,8 @@ class Surface(_SurfaceEntityBase):
     private_attribute_sub_components: Optional[List[str]] = pd.Field(
         [], description="The face ids in geometry that composed into this `Surface`."
     )
+    # pylint: disable=fixme
+    # TODO: This should be deprecated since it is not very useful or easy to use.
     private_attribute_potential_issues: List[_SurfaceIssueEnums] = pd.Field(
         [],
         description="Issues (not necessarily problems) found on this `Surface` after inspection by "

flow360/component/simulation/user_defined_dynamics/user_defined_dynamics.py

@@ -8,9 +8,6 @@
 from flow360.component.simulation.framework.entity_base import EntityList
 from flow360.component.simulation.framework.expressions import StringExpression
 from flow360.component.simulation.primitives import Cylinder, GenericVolume, Surface
-from flow360.component.simulation.validation.validation_context import (
-    get_validation_info,
-)
 from flow360.component.simulation.validation.validation_utils import (
     check_deleted_surface_in_entity_list,
 )
@@ -115,17 +112,19 @@ def ensure_surface_existence(cls, value):
     @classmethod
     def ensure_output_surface_existence(cls, value):
         """Ensure that the output target surface is not a deleted surface"""
-        validation_info = get_validation_info()
-        if validation_info is None or validation_info.auto_farfield_method is None:
-            # validation not necessary now.
-            return value
 
-        # - Check if the surfaces are deleted.
-        # pylint: disable=protected-access
-        if isinstance(value, Surface) and value._will_be_deleted_by_mesher(
-            validation_info.auto_farfield_method
-        ):
-            raise ValueError(
-                f"Boundary `{value.name}` will likely be deleted after mesh generation. Therefore it cannot be used."
-            )
+        # --- Disabled for FXC-2006
+        # validation_info = get_validation_info()
+        # if validation_info is None or validation_info.auto_farfield_method is None:
+        #     # validation not necessary now.
+        #     return value
+
+        # # - Check if the surfaces are deleted.
+        # # pylint: disable=protected-access
+        # if isinstance(value, Surface) and value._will_be_deleted_by_mesher(
+        #     validation_info.auto_farfield_method
+        # ):
+        #     raise ValueError(
+        #         f"Boundary `{value.name}` will likely be deleted after mesh generation. Therefore it cannot be used."
+        #     )
         return value

flow360/component/simulation/validation/validation_simulation_params.py

@@ -313,6 +313,10 @@ def _check_complete_boundary_condition_and_unknown_surface(
 ):  # pylint:disable=too-many-branches
     ## Step 1: Get all boundaries patches from asset cache
 
+    return params
+
+    # --- Disabled for FXC-2006
+    # pylint: disable=unreachable
     current_lvls = get_validation_levels() if get_validation_levels() else []
     if all(level not in current_lvls for level in (ALL, CASE)):
         return params

flow360/component/simulation/validation/validation_utils.py

@@ -7,7 +7,6 @@
 
 from flow360.component.simulation.entity_info import DraftEntityTypes
 from flow360.component.simulation.primitives import (
-    Surface,
     _SurfaceEntityBase,
     _VolumeEntityBase,
 )
@@ -68,25 +67,27 @@ def check_deleted_surface_in_entity_list(value):
     Check if any boundary is meant to be deleted
     value--> EntityList
     """
-    validation_info = get_validation_info()
-    if (
-        validation_info is None
-        or validation_info.auto_farfield_method is None
-        or validation_info.is_beta_mesher is True
-    ):
-        # validation not necessary now.
-        return value
-
-    # - Check if the surfaces are deleted.
-    for surface in value.stored_entities:
-        if isinstance(
-            surface, Surface
-        ) and surface._will_be_deleted_by_mesher(  # pylint:disable=protected-access
-            validation_info.auto_farfield_method
-        ):
-            raise ValueError(
-                f"Boundary `{surface.name}` will likely be deleted after mesh generation. Therefore it cannot be used."
-            )
+    # --- Disabled for FXC-2006
+    # validation_info = get_validation_info()
+    # if (
+    #     validation_info is None
+    #     or validation_info.auto_farfield_method is None
+    #     or validation_info.is_beta_mesher is True
+    # ):
+    #     # validation not necessary now.
+    #     return value
+
+    # # - Check if the surfaces are deleted.
+    # for surface in value.stored_entities:
+    #     if isinstance(
+    #         surface, Surface
+    #     ) and surface._will_be_deleted_by_mesher(  # pylint:disable=protected-access
+    #         validation_info.auto_farfield_method
+    #     ):
+    #         raise ValueError(
+    #             f"Boundary `{surface.name}` will likely be deleted after mesh generation. "
+    #             "Therefore it cannot be used."
+    #         )
 
     return value
 
@@ -96,19 +97,22 @@ def check_deleted_surface_pair(value):
     Check if any boundary is meant to be deleted
     value--> SurfacePair
     """
-    validation_info = get_validation_info()
-    if validation_info is None or validation_info.auto_farfield_method is None:
-        # validation not necessary now.
-        return value
-
-    # - Check if the surfaces are deleted.
-    for surface in value.pair:
-        if surface._will_be_deleted_by_mesher(  # pylint:disable=protected-access
-            validation_info.auto_farfield_method
-        ):
-            raise ValueError(
-                f"Boundary `{surface.name}` will likely be deleted after mesh generation. Therefore it cannot be used."
-            )
+
+    # --- Disabled for FXC-2006
+    # validation_info = get_validation_info()
+    # if validation_info is None or validation_info.auto_farfield_method is None:
+    #     # validation not necessary now.
+    #     return value
+
+    # # - Check if the surfaces are deleted.
+    # for surface in value.pair:
+    #     if surface._will_be_deleted_by_mesher(  # pylint:disable=protected-access
+    #         validation_info.auto_farfield_method
+    #     ):
+    #         raise ValueError(
+    #             f"Boundary `{surface.name}` will likely be deleted after mesh generation. "
+    #             "Therefore it cannot be used."
+    #         )
 
     return value
 

tests/simulation/params/test_automated_farfield.py

@@ -190,10 +190,11 @@ def _run_validation(params):
 
     # Valid Symmetric but did not use it
     errors = _run_validation(params)
-    assert len(errors) == 1
-    assert (
-        "The following boundaries do not have a boundary condition: symmetric." in errors[0]["msg"]
-    )
+    # --- Disabled for FXC-2006
+    # assert len(errors) == 1
+    # assert (
+    #     "The following boundaries do not have a boundary condition: symmetric." in errors[0]["msg"]
+    # )
 
     params.models.append(SymmetryPlane(surfaces=[farfield.symmetry_planes]))
     errors = _run_validation(params)