You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The dimensions of the wall generated by rectangleWall(h,w) is not the same as the dimensions of a wall generated by trapezoidWall(h, w, w).
Minimum Viable Example
See traprectissue attached - this is a generator that creates a trapezoid wall and a rectangle wall with the same parameters.
Expected Outcome
I would expect that the two walls are the same size; the trapezoid is smaller than the rectangle. The size difference means that these two wall parts can't be used in the same design without significant effort to adjust the sizes.
Screenshots or Drawings
Image:
Additional context
Of course, the other way to "fix" this is in a mixed-part design to only use trapezoidWall()s
Is the current behavior the right behavior? I don't think it is, since it does mean that you can't easily mix and match the trap walls with rect walls.
In a similar vein, is overallheight (init.py:2620) computed correctly? I think edges[2].spacing needs to be added in?
I'll try to write the patch to fix these issues.
The text was updated successfully, but these errors were encountered:
Description
The dimensions of the wall generated by rectangleWall(h,w) is not the same as the dimensions of a wall generated by trapezoidWall(h, w, w).
Minimum Viable Example
See traprectissue attached - this is a generator that creates a trapezoid wall and a rectangle wall with the same parameters.
Expected Outcome
I would expect that the two walls are the same size; the trapezoid is smaller than the rectangle. The size difference means that these two wall parts can't be used in the same design without significant effort to adjust the sizes.
Screenshots or Drawings
Image:
Additional context
Of course, the other way to "fix" this is in a mixed-part design to only use trapezoidWall()s
Is the current behavior the right behavior? I don't think it is, since it does mean that you can't easily mix and match the trap walls with rect walls.
In a similar vein, is overallheight (init.py:2620) computed correctly? I think edges[2].spacing needs to be added in?
I'll try to write the patch to fix these issues.
The text was updated successfully, but these errors were encountered: