Total float calculation now takes into account calendar

src/ifcopenshell-python/ifcopenshell/api/geometry/add_representation.py

@@ -60,14 +60,11 @@ class Usecase:
         else:
             vx = vz.cross(X_AXIS)
         vy = vx.cross(vz)
-        tM = Matrix([
-            [vx.x, vy.x, vz.x, co.x],
-            [vx.y, vy.y, vz.y, co.y],
-            [vx.z, vy.z, vz.z, co.z],
-            [0, 0, 0, 1]
-        ]).inverted()
+        tM = Matrix(
+            [[vx.x, vy.x, vz.x, co.x], [vx.y, vy.y, vz.y, co.y], [vx.z, vy.z, vz.z, co.z], [0, 0, 0, 1]]
+        ).inverted()
 
-        return any([abs((tM @ v.co).z) > threshold  for v in face.verts])
+        return any([abs((tM @ v.co).z) > threshold for v in face.verts])
 
     def evaluate_geometry(self):
         for modifier in self.settings["blender_object"].modifiers:

src/ifcopenshell-python/ifcopenshell/api/sequence/recalculate_schedule.py

@@ -242,7 +242,14 @@ class Usecase:
         elif data.get("late_start") is None:
             data["late_start"] = self.offset_date(data["late_finish"], -data["duration"], data)
 
-        data["total_float"] = data["late_finish"] - data["early_finish"]
+        if data["duration_type"] == "WORKTIME":
+            data["total_float"] = datetime.timedelta(
+                days=ifcopenshell.util.sequence.count_working_days(
+                    data["early_finish"], data["late_finish"], data["calendar"]
+                )
+            )
+        else:
+            data["total_float"] = data["late_finish"] - data["early_finish"]
 
         # Waiting for yassine to confirm relationships
         # data["free_float"] = min([self.g.nodes[s]["early_start"] for s in successors])