4 Replies Latest reply on Jun 2, 2021 11:23 AM by neilk

    gear step ratio


      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;  
            driver_pulse_state = PULSE_LOW;  
            driver_pulse_state = PULSE_IDLE;  
      void encoderISR(){  
        pulse_buffer += STEPS_PR_ENCODER_PULSE;  
        • Re: gear step ratio

          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
          3 of 3 people found this helpful
            • Re: gear step ratio

              all control by Accelstepper

              i get a done code

              and i dont now why it is not working correctly

                • Re: gear step ratio

                  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.

                  1 of 1 people found this helpful
              • Re: gear step ratio

                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.



                1 of 1 people found this helpful