brainstorm3 icon indicating copy to clipboard operation
brainstorm3 copied to clipboard

BIDS for NIRS

Open Edouard2laire opened this issue 4 years ago • 2 comments

Hello,

Recently, BIDS started to be implemented for NIRS (mainly using the snirf format, already supported by Brainstorm). I was wondering if you could guide me on what part of the code modify to import BIDS dataset containing nirs data such as : https://github.com/rob-luke/BIDS-NIRS-Tapping/tree/388d2cdc3ae831fc767e06d9b77298e9c5cd307b ?

Regards, Edouard

Edouard2laire avatar Dec 03 '21 03:12 Edouard2laire

The two files to edit would be process_import_bids.m and process_export_bids.m.

  • process_import_bids:
    • line 541: add the the modality
    • line 608: add the file extension
    • line 676: add the sensor types
    • If there are new metadata files to import, this might require some more work, otherwise it could be pretty simple
  • process_export_bids: I'm not familiar with it, I have never used it. This is only to export unmodified continuous data

Hopefully at some point, we will replace the parsing (import) and the file name generation (export) with methods called from BIDS-Matlab: https://github.com/bids-standard/bids-matlab

Le me know if you're stuck somewhere.

ftadel avatar Dec 03 '21 09:12 ftadel

Thanks. Started here: https://github.com/Edouard2laire/brainstorm3/tree/BIDS-nirs

Edit: it seems it's working on this dataset :). Will do more test monday

Edouard2laire avatar Dec 03 '21 22:12 Edouard2laire

Any update on this?

ftadel avatar Nov 11 '22 10:11 ftadel

Hello,

I have been quite busy lately but I should be able to work on it soon. First I need to make a PR to improve a bit the snirf reader. then it will be mainly copying from the previous PR.

Edouard2laire avatar Nov 11 '22 22:11 Edouard2laire

ok. Done in #589. Can't believe already one year passed since we open this thread

Edouard2laire avatar Nov 14 '22 18:11 Edouard2laire