Detect Keiser bike version and set stats timeout accordingly.

[chriselsen]

Keiser changed the broadcast interval in firmware revision 6.30 from ~ 2 seconds to ~ 0.3 seconds. As a result bikes before version 6.30 (old) need to use a higher stats timeout value of 7 seconds to detect a communication failure, while bikes starting with version 6.30 (new) can use a lower stats timeout value of 1 second.

This change determines the bikes firmware version on first discovery and sets the stats timeout value accordingly.
For unknown firmware versions (e.g. 5.12, which doesn't exist) a "safe" stats timeout value of 7 seconds is considered.

[chriselsen]

@ptx2 This should be ready for merging

[ptx2]

Do we want FW 7.00+ to use the shorter timeout too -- it looks like it will go back to the longer timeout?

[chriselsen]

There should only be a version 6.xx based on the Keiser Dev documentation. Some of their examples completely ignore any data packages that don't have a Major of 0x06.
Therefore just in case we ever encounter something like that, we treat it as unknown and play it safe with the 7 sec timeout. What do you think?

[ptx2]

That seems reasonable to me :-)

[ptx2]

Nice one!

[tompijls]

anything needing testing @chriselsen @ptx2 ?

[chriselsen]

anything needing testing @chriselsen @ptx2 ?

Sure. More testing is always a good idea. Although here the hope is that you won't notice a functional difference. As this has been merged into 'main' now, you can try this via:

To try this out:

Start with current Gymnasticon Pi image

• Setup build environment:

sudo apt-get install git
cd
git clone https://github.com/ptx2/gymnasticon.git
cd gymnasticon
sudo service gymnasticon stop
sudo setcap cap_net_raw+eip $(eval readlink -f $(which node))

• Build it and test it

npm install
npm run build
npm link
NODE_DEBUG=gymnasticon:bike* gymnasticon --bike keiser

When your Keiser bike is detected, it should show the bike's version ID as well as the internally used stats timeout. Other than that, you should not see any changes.

[tompijls]

worked fine!