start rearrange structure

seperate logic and procecces for checking
only include some check functions for BPS
e1_checkIFCIDS_dev.py example is working

Author: Cudok

bim2sim/tasks/common/check_ifc_ids.py

@@ -93,34 +93,6 @@ def run(self, ifc_files: [IfcFileClass]):
             # checks are domain specific
             # Reset class based on domain to run the right check.
             # Not pretty but works. This might be refactored in #170
-            if ifc_file.domain == IFCDomain.hydraulic:
-                self.logger.info(f"Processing HVAC-IfcCheck")  # todo
-            elif ifc_file.domain == IFCDomain.arch:
-                self.logger.info(f"Processing BPS-IfcCheck")  # todo
-                self.__class__ = CheckIfcBPS
-                self.__class__.__init__(self, self.playground)
-                self.paths = paths  # TODO needed in if loop, here need better solution
-            elif ifc_file.domain == IFCDomain.unknown:
-                self.logger.info(f"No domain specified for ifc file "
-                                 f"{ifc_file.ifc_file_name}, not processing "
-                                 f"any checks")
-                return
-            else:
-                self.logger.info(
-                    f"For the Domain {ifc_file.domain} no specific checks are"
-                    f" implemented currently. Just running the basic checks."
-                    f"")
-
-            ## begin copy form old ifc check (only tempory until new structure is working)
-            # prepare data for checking (filering)
-
-            self.sub_inst = ifc_file.file.by_type(self.sub_inst_cls)
-
-            # checking itself
-            self.error_summary_sub_inst = self.check_inst(
-                self.validate_sub_inst, self.sub_inst)
-
-            ## end copy form old ifc check (only tempory until new structure is working)
 
             # check uniqueness of GUIDs
             self.all_guids_unique, self.double_guids = CheckLogicBase.run_check_guid_unique(ifc_file)
@@ -141,55 +113,41 @@ def run(self, ifc_files: [IfcFileClass]):
             #       critical: if loaded IFC is not IFC4
             if self.version_error:
                 self.logger.critical(f"ifc Version is not fitting. Should be IFC4, but here: " + self.ifc_version)
-            # write reportes self made checks
-            base_name = f"/{ifc_file.domain.name.upper()}_" \
-                        f"{ifc_file.ifc_file_name[:-4]}"
-            self._write_errors_to_html_table(base_name, ifc_file.domain)
 
-    ###### old ifc check, maybe stay here
-    @staticmethod
-    def check_inst(validation_function: Callable, elements: list):
-        """Uses sb_validation/ports/elements functions in order to check each
-        one and adds error to dictionary if object has errors. Combines the
-        (error) return of the specific validation function with the key (mostly
-        the GlobalID).
+            # prepare data for checking (filering)
+            self.sub_inst = ifc_file.file.by_type(self.sub_inst_cls)
 
-        Args: validation_function: function that compiles all the
-        validations to be performed on the object (sb/port/instance) elements:
-        list containing all objects to be evaluates
+            if ifc_file.domain == IFCDomain.hydraulic:
+                self.logger.info(f"Processing HVAC-IfcCheck")  # todo
+            elif ifc_file.domain == IFCDomain.arch:
+                self.logger.info(f"Processing BPS-IfcCheck")  # todo
+                # checking itself
+                chlb = CheckLogicBase()
+                self.error_summary_sub_inst = chlb.check_inst(
+                    chlb.validate_sub_inst, self.sub_inst)
+                self.paths = paths  # TODO needed in if loop, here need better solution
+            elif ifc_file.domain == IFCDomain.unknown:
+                self.logger.info(f"No domain specified for ifc file "
+                                 f"{ifc_file.ifc_file_name}, not processing "
+                                 f"any checks")
+                return
+            else:
+                self.logger.info(
+                    f"For the Domain {ifc_file.domain} no specific checks are"
+                    f" implemented currently. Just running the basic checks."
+                    f"")
 
-        Returns:
-            summary: summarized dictionary of errors, where the key is the
-                GUID + the ifc_type
+            ## begin copy form old ifc check (only tempory until new structure is working)
 
-        """
-        summary = {}
-        for inst in elements:
-            error = validation_function(inst)
-            if len(error) > 0:
-                if hasattr(inst, 'GlobalId'):
-                    key = inst.GlobalId + ' ' + inst.is_a()
-                else:
-                    key = inst.is_a()
-                summary.update({key: error})
-        return summary
 
-    @staticmethod
-    def apply_validation_function(fct: bool, err_name: str, error: list):
-        """
-        Function to apply a validation to an instance, space boundary or
-        port, it stores the error to the list of errors.
 
-        Args:
-            fct: validation function to be applied
-            err_name: string that define the error
-            error: list of errors
+            ## end copy form old ifc check (only tempory until new structure is working)
 
-        """
-        if not fct:
-            error.append(err_name)
+            # write reportes self made checks
+            base_name = f"/{ifc_file.domain.name.upper()}_" \
+                        f"{ifc_file.ifc_file_name[:-4]}"
+            self._write_errors_to_html_table(base_name, ifc_file.domain)
 
-    ###### old ifc check, maybe stay here
 
     def validate_sub_inst(self, sub_inst: list) -> list:
         raise NotImplementedError
@@ -370,7 +328,6 @@ def _write_errors_to_html_table(self, base_name: str, domain: IFCDomain):
                 webbrowser.open(f"file://{out_file.buffer.name}")
 
 
-
 class CheckIfcBPS(CheckIfc):
     """
     Check an IFC file, for a number of conditions (missing information,
@@ -449,6 +406,12 @@ class CheckLogicBase():
     useful for all checking use cases.
     """
 
+    def __init__(self):
+        # used for preparing data for checking, is filder keyword
+        self.sub_inst_cls = 'IfcRelSpaceBoundary'
+        self.plugin = bps
+        self.space_ndicator = True
+
     def run_check_guid_unique(ifc_file) -> (bool, dict):
         """check the uniqueness of the guids of the IFC file
 
@@ -541,6 +504,108 @@ def run_check_ifc_version(ifc: ifcos.file) -> (bool, str):
         return (version_error, schema)
 
 
+    ###### old ifc check, maybe stay here
+    @staticmethod
+    def check_inst(validation_function: Callable, elements: list):
+        """Uses sb_validation/ports/elements functions in order to check each
+        one and adds error to dictionary if object has errors. Combines the
+        (error) return of the specific validation function with the key (mostly
+        the GlobalID).
+
+        Args: validation_function: function that compiles all the
+        validations to be performed on the object (sb/port/instance) elements:
+        list containing all objects to be evaluates
+
+        Returns:
+            summary: summarized dictionary of errors, where the key is the
+                GUID + the ifc_type
+
+        """
+        summary = {}
+        for inst in elements:
+            error = validation_function(inst)
+            if len(error) > 0:
+                if hasattr(inst, 'GlobalId'):
+                    key = inst.GlobalId + ' ' + inst.is_a()
+                else:
+                    key = inst.is_a()
+                summary.update({key: error})
+        return summary
+
+    @staticmethod
+    def apply_validation_function(fct: bool, err_name: str, error: list):
+        """
+        Function to apply a validation to an instance, space boundary or
+        port, it stores the error to the list of errors.
+
+        Args:
+            fct: validation function to be applied
+            err_name: string that define the error
+            error: list of errors
+
+        """
+        if not fct:
+            error.append(err_name)
+
+    ###### old ifc check, maybe stay here
+
+    @staticmethod
+    def _check_rel_space(bound: entity_instance):
+        """
+        Check that the space boundary relating space exists and has the
+        correct class.
+
+        Args:
+            bound: Space boundary IFC instance
+
+        Returns:
+            True: if check succeeds
+            False: if check fails
+        """
+        return any(
+            [bound.RelatingSpace.is_a('IfcSpace') or
+             bound.RelatingSpace.is_a('IfcExternalSpatialElement')])
+
+    @staticmethod
+    def _check_rel_building_elem(bound: entity_instance):
+        """
+        Check that the space boundary related building element exists and has
+        the correct class.
+
+        Args:
+            bound: Space boundary IFC instance
+
+        Returns:
+            True: if check succeeds
+            False: if check fails
+        """
+        if bound.RelatedBuildingElement is not None:
+            return bound.RelatedBuildingElement.is_a('IfcElement')
+
+    def validate_sub_inst(self, bound: entity_instance) -> list:
+        """
+        Validation function for a space boundary that compiles all validation
+        functions.
+
+        Args:
+            bound: ifc space boundary entity
+
+        Returns:
+            error: list of errors found in the ifc space boundaries
+        """
+        error = []
+        self.apply_validation_function(self._check_rel_space(bound),
+                                       'RelatingSpace - '
+                                       'The space boundary does not have a '
+                                       'relating space associated', error)
+        self.apply_validation_function(self._check_rel_building_elem(bound),
+                                       'RelatedBuildingElement - '
+                                       'The space boundary does not have a '
+                                       'related building element associated',
+                                       error)
+        return error
+
+
 class CheckLogicBPS(CheckLogicBase):
     """Provides additional logic for ifc files checking regarding BPS."""