JohK / nutsnbolts

A OpenSCAD library that allows for simple creation of nuts and bolts and respective nut catches and screw holes
GNU General Public License v3.0
267 stars 41 forks source link

Implemented pitch diameters and module to create a pitch hole #8

Closed Maaajaaa closed 4 years ago

Maaajaaa commented 7 years ago

implements #7, the module hole_pitch differs from hole_through in the fact, that it allows to use standardized hole sizes in three variation (fine, medium, coarse) instead of guessing a clearance (as you do in the hole_through).

JohK commented 7 years ago

Hi,

Thank you for your work!

I have several remarks concerning the implementation that I would like to have resolved before merging:

What's your opinion on those points?

Cheers Johannes

Maaajaaa commented 7 years ago

Hi, so here's my opinion about your remarks:

  1. Makes sense to me
  2. Ok, I didn't know and care about that but thanks for the hint
  3. Well I though the types where some kind of sane clearance defaults (I don't know the standard or engineering for that matter that well)
  4. Sounds good but I'm not sure how I'd implement the fine, medium and course "types" or rather how to name that parameter

Have a nice day Mattis

JohK commented 4 years ago

Hi Mattis,

have you put some more work into this and want to create a new pull request?

Johk