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

Higher Threading Pitch #178

Open
SamGuliver opened this issue Sep 6, 2021 · 7 comments
Open

Higher Threading Pitch #178

SamGuliver opened this issue Sep 6, 2021 · 7 comments

Comments

@SamGuliver
Copy link

I wanted to add a higher threading pitch for making customs springs.
I am not very good at coding so my attempt at simply adding missing values to the "metric_thread_table" resulted in multiple errors. Can somebody help me, finding out what I might need to change as well to add higher threading pitches?

Kind Regards

@JonWoellhaf
Copy link

JonWoellhaf commented Sep 6, 2021 via email

@SamGuliver
Copy link
Author

Hey Jon,

well thanks to you I tried out the newer 1.4 Version which seems to have now resolved my error while compiling and the threading numbers show up correctly when I use try to use them in the UI.

But have to found a different issue that seems to have existed before I did my update as well. The threads don´t go higher than 4mm/rev. I can enter 5 or 6 or higher, but the actual pitch of the resulting thread does not exceed 4mm/rev. Everything below 4 works accurately. Maybe you know whats happening?
Thank you for helping!

Kind regards

@clough42
Copy link
Owner

clough42 commented Sep 7, 2021 via email

@SamGuliver
Copy link
Author

Sure.

{ .display = {BLANK, SIX | POINT, FIVE, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(650) },
{ .display = {BLANK, SEVEN, BLANK, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(700) },
{ .display = {BLANK, SEVEN | POINT, FIVE, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(750) },
{ .display = {BLANK, EIGHT, BLANK, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(800) },
{ .display = {BLANK, EIGHT | POINT, FIVE, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(850) },
{ .display = {BLANK, NINE, BLANK, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(900) },
{ .display = {BLANK, NINE | POINT, FIVE, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(950) },
{ .display = {ONE, ZERO, BLANK, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(1000)},
{ .display = {ONE, ZERO | POINT, FIVE, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(1050)},
{ .display = {ONE, ONE, BLANK, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(1100)},
{ .display = {ONE, ONE | POINT, FIVE, BLANK}, .leds = LED_THREAD | LED_MM, HMM_FRACTION(1150)},

These are the ones I added to the Tables.cpp file. But as I said I am experiencing issues with the unaltered code as well (above 4mm and above a certain range in the imperial threading too. As I am turning the chuck by hand, the calculations not keeping up, or the stepper being too slow, should not be an issue).

Thank you.

@blackfister
Copy link

Hi Sam, did you manage to get coarser threads?
If so, the coding posted should be copied into tables, at what line, if I may ask. Thank you

@johnsonm
Copy link

See my PR #255 which does three things:

  • Adds, optionally, a lot more thread specifications
  • Adds the ability for new thread pitches not supported by the previous configuration (one more decimal digit for both metric and imperial pitches, allowing pitches like .125mm and 5.75tpi)
  • Makes it easier to select the default feeds/speeds at startup

Sadly I'm still building the hardware so I haven't been able to test those pitches on my lathe yet, but I'll be done fairly soon. ☺

@johnsonm
Copy link

That took longer than I expected due to having to redesign the hardware, but all the pitches I tested so far in #255 have been correct.

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

No branches or pull requests

5 participants