gear step ratio

Question

Hello everyone help to understand the code need to increase pulse_buffer in 1.2 but the code only allows integer 1 2 and so on. tried changing the pulse_buffer datatype to round off STEPS_PR_ENCODER_PULSE but it only works equally well with the whole. can still some parameter increase to then divide? or how it make by another way because its impossible set impulse 1.2 #define ENCODER_PULSE_PIN 2
#define ENCODER_DIR_PIN 3
#define DRIVER_STEP_PIN 4
#define DRIVER_DIR_PIN 7
#define DRIVER_EN_PIN 8

#define STEPS_PR_ENCODER_PULSE 1
#define DRIVER_PULSE_PERIOD_US 1

enum Driver_pulse_state_enum {PULSE_IDLE, PULSE_HIGH, PULSE_LOW};

uint8_t dir = LOW; 
uint16_t pulse_buffer = 0; 
unsigned long time_now = 0; 
uint16_t driver_pulse_hold_time_us = DRIVER_PULSE_PERIOD_US/2; 
uint8_t driver_pulse_state = PULSE_IDLE;

void setup() {

pinMode(ENCODER_PULSE_PIN, INPUT); 
 pinMode(ENCODER_DIR_PIN, INPUT); 
 pinMode(DRIVER_STEP_PIN, OUTPUT); 
 pinMode(DRIVER_DIR_PIN, OUTPUT); 
 pinMode(DRIVER_EN_PIN, OUTPUT);

attachInterrupt(digitalPinToInterrupt(ENCODER_PULSE_PIN), encoderISR, RISING);

digitalWrite(DRIVER_EN_PIN, HIGH);

void loop() {

if((pulse_buffer) && (driver_pulse_state == PULSE_IDLE)){ 
 driver_pulse_state = PULSE_HIGH; 
 digitalWrite(DRIVER_DIR_PIN, digitalRead(ENCODER_DIR_PIN)); 
 digitalWrite(DRIVER_STEP_PIN, HIGH); 
 time_now = micros(); 
 }

if((micros() - time_now > driver_pulse_hold_time_us) && (driver_pulse_state == PULSE_LOW)){ 
 driver_pulse_state = PULSE_HIGH; 
 digitalWrite(DRIVER_STEP_PIN, HIGH); 
 time_now = micros(); 
 }

if((micros() - time_now > driver_pulse_hold_time_us) && (driver_pulse_state == PULSE_HIGH)){ 
 digitalWrite(DRIVER_STEP_PIN, LOW); 
 time_now = micros();

pulse_buffer -= 1; 
if(pulse_buffer < 0){ 
 pulse_buffer = 0; 
 }

if(pulse_buffer){ 
 driver_pulse_state = PULSE_LOW; 
 } 
else{ 
 driver_pulse_state = PULSE_IDLE; 
 } 
 }

}

void encoderISR(){ 
 pulse_buffer += STEPS_PR_ENCODER_PULSE; 
}

fmilburn · Answer

Do you want to use floating point? You have not provided a lot of information on what you are trying to do and floating point seems unusual with an encoder but here are some thoughts: pulse_buffer is in an ISR, it should be declared volatile pulse_buffer is unint16_t so it will be treated as an integer unless it is declared float possibly a cast could be used but it isn’t clear what the objective is

neilk · Answer

I agree with fmilburn

Get the library example working first and then start changing things to make the software do what you want. If/when it stops working, then you've got a problem with the most recent change!

To help us help you it would be helpful to see your wiring diagram and an to have a clearer explanation of what you are trying to achieve.

Neil

fmilburn · Answer

I don’t see AccelStepper in the sketch above so it is not possible to help with that. The problem could be wiring, a problem in the code that isn’t listed above, damaged parts, or any number of other things. Much more information would be needed. First though, rule out everything you can and narrow down the issue. If it is software then get the Arduino library example working and determine what change makes it break. Same way with hardware, get the individual pieces working and determine one at a time what addition or piece makes it break.

gear step ratio

Top Replies