Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-08-10 01:41:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixed bug in shape_builder.mep_transition_calculate

Browse Source 
it wasn't taking account that `h` can also end up negative
This commit is contained in:
Andrej730
2023-08-24 18:16:02 +05:00
parent 2b1e17681f
commit a30ad9b886
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcopenshell-python/ifcopenshell/util/shape_builder.py
+9 -6
View File
@@ -1160,10 +1160,11 @@ class ShapeBuilder:
return None
h = (a + b + sqrt(h0)) / (2 * t)
if h < abs(offset.y) or is_x(h, offset.y):
length_squared = h**2 - offset.y**2
if length_squared <= 0:
print(f"B. angle = {angle} requires h = {h} which is not possible with y offset = {offset.y}")
return None
length = sqrt(h**2 - offset.y**2)
length = sqrt(length_squared)
if verbose:
A = (end_half_dim if end_profile else start_half_dim) * V(1, 0, 0)
@@ -1178,10 +1179,11 @@ class ShapeBuilder:
if is_x(offset.x, 0):
angle = 90 # NOTE: for now we just hardcode the good value for that case
h = start_half_dim.x / tan(radians(angle / 2))
if h < abs(offset.y) or is_x(h, offset.y):
length_squared = h**2 - offset.y**2
if length_squared <= 0:
print(f"B. angle = {angle} requires h = {h} which is not possible with y offset = {offset.y}")
return None
length = sqrt(h**2 - offset.y**2)
length = sqrt(length_squared)
if verbose:
O = V(0, 0, 0)
@@ -1191,10 +1193,11 @@ class ShapeBuilder:
print(f"B. length = {length}, requested angle = {angle}, tested angle = {tested_angle}")
else:
h = offset.x / tan(radians(angle))
if h < abs(offset.y) or is_x(h, offset.y):
length_squared = h**2 - offset.y**2
if length_squared <= 0:
print(f"C. angle = {angle} requires h = {h} which is not possible with y offset = {offset.y}")
return None
length = sqrt(h**2 - offset.y**2)
length = sqrt(length_squared)
if verbose:
A = V(-start_half_dim.x, 0, 0)