bcf - remove lru_cache in build_viewpoint #4857

Andrej730 · Andrej730 · commit 1c95f6cc3f82 · 2024-08-09T19:00:10.000+05:00
disabling it as it currently won't allows users to create a different viewpoint for the same IFC element (in the same or other bcf file) and also entity position can change between `build_viewpoint` calls
diff --git a/src/bcf/bcf/v2/visinfo.py b/src/bcf/bcf/v2/visinfo.py
@@ -1,6 +1,5 @@
 import uuid
 import zipfile
-from functools import lru_cache
 from typing import Any, Iterable, Optional
 
 import numpy as np
@@ -206,7 +205,6 @@ def create_from_point_and_guids(
         )
 
 
-@lru_cache(maxsize=None)
 def build_viewpoint(element: entity_instance) -> mdl.VisualizationInfo:
     """
     Return a BCF viewpoint of an IFC element.
diff --git a/src/bcf/bcf/v3/visinfo.py b/src/bcf/bcf/v3/visinfo.py
@@ -1,6 +1,5 @@
 import uuid
 import zipfile
-from functools import lru_cache
 from typing import Any, Iterable, Optional
 
 import numpy as np
@@ -206,7 +205,6 @@ def create_from_point_and_guids(
         )
 
 
-@lru_cache(maxsize=None)
 def build_viewpoint(element: entity_instance) -> mdl.VisualizationInfo:
     """
     Return a BCF viewpoint of an IFC element.
