openscad / MCAD

OpenSCAD Parametric CAD Library (LGPL 2.1)
http://reprap.org/wiki/MCAD
578 stars 192 forks source link

f/threads: Update to file version 2.5 #80

Closed rockstorm101 closed 3 years ago

rockstorm101 commented 3 years ago

Obtained from: https://dkprojects.net/openscad-threads/threads.scad

Fixes openscad/MCAD#79

hyperair commented 3 years ago

I think the two files have diverged significantly, so we shouldn't just blindly check-in the new version from openscad-threads. We can try to merge in some features, but otherwise, the library in MCAD should be considered a complete fork.

Some main differences that aren't easily reconciliable:

hyperair commented 3 years ago

I'm closing this PR for now. Please open another if you wish to port the new features over from openscad-threads.

rockstorm101 commented 3 years ago

Hi @hyperair, thank you for reviewing this. I agree with your comments, particularly:

I still don't think they should be part of the same module though.

and

we use sweep() of a 2D object for a much more efficient implementation of threads [...] We also obey $fn, $fs, and $fa, which the other version does not.

So I'm happy to close this. I'll try and work out (hopefully with Dan) how to re-implement all this.

EDIT: Did I miss any other module more mature to produce threads?