Arduino-PID-Library icon indicating copy to clipboard operation
Arduino-PID-Library copied to clipboard

Use float instead of double?

Open flannelhead opened this issue 9 years ago • 0 comments

Is there any particular reason to use double as the floating point data type instead of float? I know it won't matter much since they are the same on the Arduino, but using this library kind of forces one to write all the other related arithmetic in terms of doubles. Considering that the Arduino platform now supports many more microcontrollers than just AVRs, it would be wise to change to floats.

I realize this change would break a good deal of existing code making use of this library. Hence, I'm willing to fork the library if this change is not acceptable in the upstream.

As a side note, I'd like to make a pull request in order to fix the indentation in the source code. I don't feel like opening a separate issue for this, so I'm asking what your preferred indentation level is (also, spaces or tabs).

Thank you for the great library!

flannelhead avatar Feb 12 '16 09:02 flannelhead