-
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
Small fixes [for case of packet fragmentation] #102
Conversation
Current coverage is 93.53% (diff: 100%)@@ 1.4.0 #102 diff @@
==========================================
Files 4 4
Lines 2180 2181 +1
Methods 126 126
Messages 0 0
Branches 475 475
==========================================
+ Hits 2039 2040 +1
Misses 141 141
Partials 0 0
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So these are the tests for #101 ? If yes, why not combine both PRs? Curious on this one and also confused on the order I'm supposed to merge.
@@ -1597,6 +1597,8 @@ function OpenBCIFactory () { | |||
this.curParsingMode = k.OBCIParsingNormal; | |||
this.buffer = null; | |||
this.emit('ready'); | |||
} else { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good!
I'm surprised this passed tests because fragmentation is not in master |
will rebase onto #100 |
These adjustments allow the tests to pass if #101 is merged or used.
Incidentally also fixes some initialization issues when communicating with a real live board.