Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rectangleWall and trapezoidWall do not make the same wall for the same parameters #736

Open
mjhandel opened this issue Dec 13, 2024 · 0 comments
Labels

Comments

@mjhandel
Copy link

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: Screenshot 2024-12-12 at 9 03 46 PM

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.

@mjhandel mjhandel added the bug label Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant