-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tracking: Ganglion support #80
Comments
@baffo32 any thoughts on how to properly bring in ganglion code to the repo? See https://github.com/aj-ptw/simblee-js/blob/simulator/openBCIGanglion.js for the current node driver. There is going to have to be some major test writing to get coverage. I think there needs to be an const ganglion = require('openbci').ganglion; I think they [openbci] are working on a name for the 32 bit board, but still the 32bit board with import |
Ganglion looks very different at first glance. I'd definitely at least separate the implementations into distinct, separate files; separate folders if openbci might be making more board variants. The two different drivers should provide the same function names as each other, taking the same function arguments, when they have functions that do similar things. They should dispatch the same set of events. (Later somebody could write a single 'openbci' driver which calls out to the others to handle all boards, and the user wouldn't have to care what kind of board was connected at all.) If any code would be duplicated, it should be pulled out into a common base class or utilities file, so it may be re-used. I think it could be good to handle very different firmware versions as separate drivers, too. |
I have started writing this code over at my own local branch. lots of fun! so many tests to write though. |
Looking to do something like this to remove the factory from the module. https://github.com/EmergingTechnologyAdvisors/node-serialport/blob/master/lib/serialport.js |
Here is what is looks like to use the ganglion code: https://github.com/OpenBCI/OpenBCI_Ganglion_Electron/blob/master/src/app.js |
Update: This will not be merged with this repo until the Ganglion driver is stable. For the ganglion code please see here: |
Adds support for ganglion.
The text was updated successfully, but these errors were encountered: