Proposed features
Emit warning messages to the log and all TCP sessions once, on the case: any of the max_v, max_i_pos, max_i_neg of any channel is set to zero+-EPSILON by pwm…
What does the graph represent? The axes aren't even labeled and don't have units...
As mentioned in the code comments, it's a comparison of qudratic and log regressions against real 80% tacho…
Well, they are subject of change, but they come from the power formula P=I^2 * R
As I said before the device should enter a failsafe mode with output currents at zero when a fan failure is detected.
Next time
parse_with_trailing_whitespaces_and_check_equality? I think 7 words is too much, and still it is a bit confusing
Well, that's pretty silly behavior. Maybe modify it to require reboot to align with Zynq, and then document.
Yes, as shown in test cases, it accepts multiple spaces and tabs
No, before this change different commands had different reaction on ending whitespace(s)
Yes, with current version of library, TIM8 can be used with the PWM pins only, and tieing tacho pin would move it to the Timer, and therefore couldn't be used without unsafe hacks. I am not sure…
That there are some signals being registered, but in general it so low so that it can be zero on some small timeline. It tipically happens on very low fan speed (from 1 to 3 included), and that…