-
Notifications
You must be signed in to change notification settings - Fork 683
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
Incorrect std value in LFP viewer #615
Comments
Here is one of the files if you'd like to test |
Thanks, I see what the problem is – the standard deviation is being calculated on the "screen buffer" (which aggregates multiple samples into a single pixel), rather the "display buffer" (which contains the original samples). We are planning to overhaul this part of the LFP Viewer in the next release (and likely switch to displaying RMS values, instead of standard deviation). We will make sure this gets fixed! |
Thank you very much for your insights! Computing RMS or MAD values on several channels at the same time would be a very exciting feature indeed! In the meantime, is there an easy way to replace the screen buffer by the display buffer in the code (like rewriting getMean and getStd with |
I think actually just changing those functions to read from the |
Thanks for your help! I updated the code accordingly florian6973@e5d92ba. It seems to work almost well:
Do you have any idea of the reason of this behavior please? Thanks! |
I'm not sure why this is happening – you could try creating a breakpoint or adding some debug outputs (using |
Thank you for your help!
But I am now even more confused:
One solution would be to use fewer points (like 200), but the estimate may be less precise. Or to understand better how the buffer works :) If you have any insights, I would be very grateful, thanks! |
I see what the issue is. If the current index is less than 2000, it fails because some of the requested data will appear at a negative index. In that case, you'll need to read from the buffer in two steps. For example, if the current index is 500, you would read 1500 values from the end of the buffer, then 500 values from the start of the buffer. Can you try implementing that and see if it fixes the problem? |
Thank you so much for your help! It fixes the problem indeed :) I just hope I will use this modified version of OpenEphys from now on, until the new release of OE. I opened a PR if you are interested. |
Thanks! We will test out the PR and merge if everything looks good |
Hi!
I generated a white noise signal with a std of 10. When playing it in OE, I get absurd std values like 1-2 instead of something around 10 (see image below).
Am I missing something please?
I thank you in advance for your help,
Best,
The text was updated successfully, but these errors were encountered: