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

MRG: Add NIRSport V2 native and snirf measurement #86

Merged
merged 3 commits into from
May 19, 2021

Conversation

rob-luke
Copy link
Member

@rob-luke rob-luke commented May 6, 2021

DO NOT MERGE I will update the reader for this file type and add tests after #9348 is merged.

Add test measurement taken with a NIRx NIRSport2 device. https://nirx.net/nirsport

Below are the screenshots taken during creation of the montage and of the actual data collected. The 3d optode positions can be seen in the screen grabs and can be used to test the reader.

This measurement also includes accelerometer data acquired simultaneously by the device. This is becoming common for fNIRS devices.

I set the device to save as both native and snirf file. So have added both. We can validate against each other to check both implementations are equivalent.

The data is quite different to previous NIRx devices. E.g. it uses json rather than structured text files.

Measurement description

Screenshot 2021-05-05 075943

Screenshot 2021-05-05 075958

Screenshot 2021-05-05 080015

Screenshot 2021-05-05 080356

Screenshot 2021-05-05 080418

Screenshot 2021-05-05 080455

Screenshot 2021-05-05 080559

Screenshot 2021-05-05 080632

@rob-luke
Copy link
Member Author

rob-luke commented May 6, 2021

Just an FYI. But after this device, MNE will support all products from this vendor.

@rob-luke
Copy link
Member Author

rob-luke commented May 6, 2021

Cross reference mne-tools/mne-python#9371

@rob-luke rob-luke changed the title DO NOT MERGE: Add NIRSport V2 native and snirf measurement MRG: Add NIRSport V2 native and snirf measurement May 19, 2021
@rob-luke
Copy link
Member Author

@larsoner could you please review and merge if happy. I am happy the files are useful for testing over at mne-tools/mne-python#9401

@larsoner larsoner merged commit 659dfb8 into mne-tools:master May 19, 2021
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

Successfully merging this pull request may close these issues.

None yet

2 participants