openbci-archive/OpenBCI_NodeJS

impedance testing not operating properly

Closed this issue · 12 comments

The values that come back from an electrode are too high. There is an issue.

Well @andrewheusser i wanted to add @teonbrooks here too. I need your help with this! So I have it mainly working, on my dev branch. However all I need to do is taken an FFT at 31.5Hz and use the intensity at that bin as the voltage. I would really love some help with this and this seems right up your guys wheel house. I can teach you how to do the automated testing to get it working and such too!

EDIT: The first sentence I meant as I wanted to add Teon as an assignee haha

@pushtheworldllc : going in tomorrow? lets take a look then

No unfortunately I am heading down to southern jersey for easter weekend but I will be online and can definitely have a google hangout!

@pushtheworldllc hey I need a little more context here...where do we want to add an FFT?

We want to use an FFT to get the amplitude of the channel voltage at 31.5Hz because that’s the frequency of the impedance test signal. Does that make sense?

AJ

From: Andy Heusser notifications@github.com
Reply-To: OpenBCI/openbci-js-sdk reply@reply.github.com
Date: Wednesday, March 30, 2016 at 6:12 AM
To: OpenBCI/openbci-js-sdk openbci-js-sdk@noreply.github.com
Cc: Andrew Keller pushtheworldllc@gmail.com
Subject: Re: [OpenBCI/openbci-js-sdk] impedance testing not operating properly (#28)

@pushtheworldllc hey I need a little more context here...where do we want to add an FFT?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Where we want to add it is in the calculation of impedances!

From: Andy Heusser notifications@github.com
Reply-To: OpenBCI/openbci-js-sdk reply@reply.github.com
Date: Wednesday, March 30, 2016 at 6:12 AM
To: OpenBCI/openbci-js-sdk openbci-js-sdk@noreply.github.com
Cc: Andrew Keller pushtheworldllc@gmail.com
Subject: Re: [OpenBCI/openbci-js-sdk] impedance testing not operating properly (#28)

@pushtheworldllc hey I need a little more context here...where do we want to add an FFT?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@pushtheworldllc i understand that, can you point me to the point in the code where it should be implemented?

https://github.com/pushtheworldllc/openbci-js-sdk/blob/dev/openBCISample.js#L215-L237

Please fork my branch if possible! There are a bunch of changes, getting ready for a 0.3

ok will do...hmmm, I'm not sure it makes sense to put it here since an FFT requires multiple time points to compute. How will computing an FFT help with the impedance estimates anyway?

See PR #32 for version 0.3.0 for major changes leading to the resolution of this bug. Need to test.

Reopening issue due to #131

Final going to close this soon