stk/min never has a value

[Double-A-92]

I'm using this with the ESPRowingMonitor (in FTMS mode) and the SPM are always at "-- stk/min".

The other values are displayed correctly. Also the SPM value is correct in other apps.

[Abasz]

Thanks for reporting. Division by zero causing NaN poisoning the sum....

When stroke rate is 0 (rower at rest, between strokes, or very slow rowing):

• value.getUint8(2) = 0
• (60 / (0 / 2)) = (60 / 0) = Infinity
• deltaStrokeTime = Infinity or NaN
• this.strokeTime becomes corrupted
• Strokes per minute calculations downstream NaN

The existing tests always used non-zero stroke rates, so they never triggered the division by zero. I will also add a new test case that specifically tests FTMS mode with stroke rate = 0 to prevent regression.

[Abasz]

Also as a side note, this issue did not surface during actual hardware testing because ESPRM incorrectly broadcasted in stop state max SPM (127) so the calculation did not result in NaN but rather in zero (correctly) which was due to the (currentStrokeCount - previousStrokeCount) being zero for the multiplication. And this was actually the reason this max SPM broadcast was not detected in ESPRM because the GUI hided it :)