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
I'm unsure if this is intentional or not but it threw me for a loop when I was coding up a parametric assembly.
The behavior of boxes is to fit their dimensions precisely including the tab cutouts. The square sheets don't. They're offset by a sheet thickness in two dimensions from 0 rather than translated to compensate. Personally I'd prefer all the functions to have the behavior of the box and once created have the 0 offset; but either case being consistent would do for regular behavior.
I realize this might be intentional for some reason but felt I should point it out to prevent someone having a similar head-scratching moment as their assembly is subtly off :). I can submit a patch if you'd like but it'd probably take me a while.
Thanks for sharing this library by the way. It's a nice time saver.
By then I might have it fixed. Shortly after posting I noticed it was a little too simplistic.
I think the method is sound but I need to follow the logic of the program a little more and clean up where the jiggering happens. I should be able to follow up with a correctly working patch!
I'm unsure if this is intentional or not but it threw me for a loop when I was coding up a parametric assembly.
The behavior of boxes is to fit their dimensions precisely including the tab cutouts. The square sheets don't. They're offset by a sheet thickness in two dimensions from 0 rather than translated to compensate. Personally I'd prefer all the functions to have the behavior of the box and once created have the 0 offset; but either case being consistent would do for regular behavior.
I realize this might be intentional for some reason but felt I should point it out to prevent someone having a similar head-scratching moment as their assembly is subtly off :). I can submit a patch if you'd like but it'd probably take me a while.
Thanks for sharing this library by the way. It's a nice time saver.
in example:
The text was updated successfully, but these errors were encountered: