Closed SFEMark closed 4 years ago
A user on the forum brought up this issue with several typos in Example 3. Here is the list of the errors with fixes:
int error = bioHub.setAlgoRange(algoRange));
int algoVal = bioHub.readAlgoRange());
int stepVal = bioHub.readAlgoStepSize());
int senVal = bioHub.readAlgoSensitivity());
int sampVal = bioHub.readAlgoSamples());
All of these lines just have one extra bracket after the function call. Editing to remove the extra bracket is all that is needed.
This does not have a call to bioHub prior to the configBpm function. Modifying to bioHub.configBpm(MODE_ONE); fixes the error.
bioHub.configBpm(MODE_ONE);
Can we get this example updated and, if necessary, push a new version of the library with these fixes? Thanks!
Fixed, thanks Mark!
Commit: https://github.com/sparkfun/SparkFun_Bio_Sensor_Hub_Library/commit/577a561f84a5c9650186982ad34dfae9bde02de2 Release: https://github.com/sparkfun/SparkFun_Bio_Sensor_Hub_Library/releases/tag/v1.0.2
edspark
commented
4 years ago edspark
commented
4 years ago
A user on the forum brought up this issue with several typos in Example 3. Here is the list of the errors with fixes:
int error = bioHub.setAlgoRange(algoRange));int algoVal = bioHub.readAlgoRange());int stepVal = bioHub.readAlgoStepSize());int senVal = bioHub.readAlgoSensitivity());int sampVal = bioHub.readAlgoSamples());All of these lines just have one extra bracket after the function call. Editing to remove the extra bracket is all that is needed.
This does not have a call to bioHub prior to the configBpm function. Modifying to
bioHub.configBpm(MODE_ONE);fixes the error.Can we get this example updated and, if necessary, push a new version of the library with these fixes? Thanks!