Skip to content
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

Bug in number of samples? #37

Open
ftadel opened this issue Nov 25, 2021 · 4 comments
Open

Bug in number of samples? #37

ftadel opened this issue Nov 25, 2021 · 4 comments

Comments

@ftadel
Copy link

ftadel commented Nov 25, 2021

A user of the Brainstorm software reported an error related with the reading of this file:
https://drive.google.com/file/d/1emCpRUogef9nI3-o2jSQCHI_EQh2gtJW/view?usp=sharing

The error is initially documented here:
https://neuroimage.usc.edu/forums/t/error-in-convert-raw-to-lfp/31468/8

Here is how to reproduce the behavior I don't understand (Win10):

DataFile = 'C:\...\G38-1.ns6';
rec = openNSx(DataFile, 'noread', 'nozeropad');
nSamples = rec.MetaTags.DataPoints;    % 70943612

rec = openNSx('read', DataFile , 'channels', 1, 'sample', 't:1:70943612', 'p:short', 'uV');
nSamplesRead = size(rec.Data,2);  % 70943714

Why does the second call to openNSx return 70943714 samples instead of 70943612?

@ftadel
Copy link
Author

ftadel commented Nov 25, 2021

PS: the version of the library used in this example is the last stable release:
https://github.com/BlackrockMicrosystems/NPMK/archive/refs/tags/5.5.2.0.zip

@kianabc
Copy link
Contributor

kianabc commented Nov 30, 2021

Hi, Thanks for reporting this. I'll check it out later this week.

Best, Kian

@kianabc
Copy link
Contributor

kianabc commented Dec 3, 2021

Hi Francois,

I tried to access the data file, but got an error message:
Screen Shot 2021-12-03 at 12 44 47 PM

Could you please ask the user to upload the problematic data file again? I'm OK with the 5 GB file.

Thanks,
Kian

@ftadel
Copy link
Author

ftadel commented Dec 6, 2021

I'm sorry, our user have deleted the file before you could download it.
On the Brainstorm forum, I asked to make it available. I'll let you know when the file can be downloaded again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants