jmharvey1 / arduino-cnc-speed-control Goto Github PK

Hello, I'm using a hall effect sensor and 2 magnets (for balance) on my spindle. I've changed line 285 to "calcrpm = (SampleCnt*30000000)/period;" to reflect the doubling of the counts and it appears to be setting the speed correctly as checked against my frequency function on my multimeter. I've noticed that when the speed gets over 10000rpm( I've changed the values for trgtRPM) the Actl RPM starts from 5000 again. I've looked for the calculation to correct this but have been unsuccessful. Could you point me in the write direction please.
Cheers
Mike

Hi jmharvey1,
Can you set the maximum speed to 20,000 rpm?
Because my spindle to a maximum speed of 18000 rpm.

Hi, and thanks for the code, just what I was looking for. This is a real nitpick but there is a compile time error for line 344 which can be corrected by changing one of the numbers to a long number ie.
long MaxRunTime = 606010L;// number of seconds the program is allowed to run be
Cheers
Mike

Hello, I've noticed that the spindle does a reset approx. every 5 minutes. I've tried to debug this but have not had any luck. It appears to be in the loop that starts at line 386. Any help would be greatly appreciated.
Cheers
Mike

i have just put this together, and it spins the spindle when turned on, and the sensor looks to be working with the scan screen etc, but if i press the select button, the app crashes and i have to power cycle to restore.

Any ideas?

C:\Users\rick\AppData\Local\Temp\build5221026074079849357.tmp\LCDKeypad\LCDKeypadR1.cpp.o: In function `LCDKeypad::LCDKeypad()':

C:\Users\rick\Documents\Arduino\libraries\LCDKeypad/LCDKeypadR1.cpp:12: multiple definition of `LCDKeypad::LCDKeypad()'

C:\Users\rick\AppData\Local\Temp\build5221026074079849357.tmp\LCDKeypad\LCDKeypad.cpp.o:C:\Users\rick\Documents\Arduino\libraries\LCDKeypad/LCDKeypad.cpp:12: first defined here

C:\Users\rick\AppData\Local\Temp\build5221026074079849357.tmp\LCDKeypad\LCDKeypadR1.cpp.o: In function `LCDKeypad::LCDKeypad()':

C:\Users\rick\Documents\Arduino\libraries\LCDKeypad/LCDKeypadR1.cpp:12: multiple definition of `LCDKeypad::LCDKeypad()'

C:\Users\rick\AppData\Local\Temp\build5221026074079849357.tmp\LCDKeypad\LCDKeypad.cpp.o:C:\Users\rick\Documents\Arduino\libraries\LCDKeypad/LCDKeypad.cpp:12: first defined here
I receive the following error message when doing a verify compile. I download your code, and then perform the compile. This is after I downloaded and installed the modified libraries. Once I get this to compile, I will then go through and change the commented out lines so that I can run this on a UNO.

Thank you for any assistance.
Rick Harms
[email protected]

C:\Users\rick\AppData\Local\Temp\build5221026074079849357.tmp\LCDKeypad\LCDKeypadR1.cpp.o: In function `LCDKeypad::LCDKeypad()':

C:\Users\rick\Documents\Arduino\libraries\LCDKeypad/LCDKeypadR1.cpp:12: multiple definition of `LCDKeypad::button()'

C:\Users\rick\AppData\Local\Temp\build5221026074079849357.tmp\LCDKeypad\LCDKeypad.cpp.o:C:\Users\rick\Documents\Arduino\libraries\LCDKeypad/LCDKeypad.cpp:12: first defined here

collect2.exe: error: ld returned 1 exit status

Error compiling.