thalmiclabs / myo.js

Myo javascript bindings
Apache License 2.0
198 stars 63 forks source link

add timestamp to EMG API (backwards compatible) #8

Closed superkruger closed 9 years ago

smngreenberg commented 9 years ago

Thanks for the PR! I think we'll want to uniformly include the timestamp for all events (not just EMG) before merging this in. Would you mind updating?

We'll also update the README when this is in.

superkruger commented 9 years ago

No prob Scott,

I'll update both.

Jaco

On Tue, Mar 31, 2015 at 7:28 AM, Scott Greenberg notifications@github.com wrote:

Thanks for the PR! I think we'll want to uniformly include the timestamp for all events (not just EMG) before merging this in. Would you mind updating?

We'll also update the README when this is in.

— Reply to this email directly or view it on GitHub https://github.com/thalmiclabs/myo.js/pull/8#issuecomment-87822347.

Jaco Kruger Email: superkruger@gmail.com Cell: +61(0)481 084 750 Skype: super.kruger

superkruger commented 9 years ago

Hey Scott, not sure if the last commit to this PR was seen 8 days ago...

smngreenberg commented 9 years ago

Woops, sorry! Merging now, thanks again.

superkruger commented 9 years ago

Small addition :) but loving the myo more and more...

On Thursday, April 9, 2015, Scott Greenberg notifications@github.com wrote:

Woops, sorry! Merging now, thanks again.

— Reply to this email directly or view it on GitHub https://github.com/thalmiclabs/myo.js/pull/8#issuecomment-90935673.

Jaco Kruger Email: superkruger@gmail.com Cell: +61(0)481 084 750 Skype: super.kruger

smngreenberg commented 9 years ago

Glad to hear it! :)