openbci-archive / OpenBCI_NodeJS

Node.js SDK for the all OpenBCI Biosensor Boards
https://www.npmjs.com/package/openbci
137 stars 52 forks source link

Some samples dropped when received in a large clump. #115

Closed baffo32 closed 7 years ago

baffo32 commented 7 years ago

This is resulting in occasional test timeouts, when samples are processed too infrequently to finish in time.

See the extra debugging output at https://travis-ci.org/baffo32/OpenBCI_NodeJS/jobs/171760737#L338 : the system receives many samples (instances of "a0" in the debug bytes), but processes only two of them ("sample 1", "sample 2").

andrewjaykeller commented 7 years ago

This may be the same error as #119

baffo32 commented 7 years ago

After reviewing the Daisy behavior as determined by the firmware source code I've found that my perceptions here were wrong. The samples are all processed correctly. I'm sorry for reporting this as an issue of dropped samples. It is probably actually an issue of being able to meet the requested samplerate under the extra overhead of the new promise instrumentation.