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

Lathe difference broken #32

Closed
jockbert opened this issue Aug 9, 2016 · 6 comments
Closed

Lathe difference broken #32

jockbert opened this issue Aug 9, 2016 · 6 comments

Comments

@jockbert
Copy link

jockbert commented Aug 9, 2016

Using Group rotating on active sketch (Shift+L) together with using solid model as difference seem to be broken.

Way to reproduce:

  1. Extrude a rectangle to a box.
  2. Sketch e.g. a circle on one of the box sides.
  3. Group rotate circle sketch (Shift+L) and use difference.

I'm using SolveSpace version 2.1~dce90646 on Linux Mint 17.3.

failed lathe
failed lathe.slvs.zip

BTW: Thanks for the effort to provide a free and usable 3D CAD program for Linux (too). The best alternative, OpenSCAD, only works for some use cases. Truly appreciated.

@whitequark
Copy link
Contributor

This is a known limitation with the SolveSpace NURBS backend. There are no plans to improve the backend for now. Check the "Force NURBS surfaces to triangle mesh" option in the group properties every time this issue arises, as a workaround.

@whitequark
Copy link
Contributor

whitequark commented Aug 9, 2016

Generally, you can achieve some success with getting the NURBS operation to proceed by making sure the NURBS surfaces do not coincide with each other (the torus is constructed from 4*4 surfaces, a circle being constructed from four rational Bezier curves). I.e. in this case, try making your torus's center slighly off the edge of the rectangle. This is, of course, inconvenient to do.

@ruevs
Copy link
Member

ruevs commented Jun 23, 2020

@whitequark would you please label this 'NURBS' so I can find it easier some day in the future.

@whitequark
Copy link
Contributor

done! feel free to send me any other issues to tag, I appreciate this

@phkahler
Copy link
Member

This is actually fixed now.

@ruevs
Copy link
Member

ruevs commented Oct 15, 2020

Fixed on 2020-10-09 22:10:22 in commit d72eba8
Pull request #733

I am adding this comment just so that things are cross-referenced.

This is the oldest "NURBS" issue on GitHub and now it is fixed - thanks @phkahler :-)
@whitequark I hope you'll enjoy this :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants