getRPM() uses hard coded number of pulses per rotation and ignores LatchMode

[ftjuh]

Hi Matthias and thanks for a great library.

If I'm not completely mistaken, getRPM() assumes an encoder with 20 pulses per Rotation:

return 60000.0 / ((float)(t * 20));

So without being able to set this number ourselves, this will be wrong in most cases. Also, I assume that LatchMode needs to be taken into account.

Jan

[mathertel]

@ftjuh : right analysis. Please create a pull request and check with your encoder.

[ftjuh]

Sorry, but my project is long finished and I moved on. I would have to start entirely from scratch to be able to contribute and test anything useful.