aces / EEG2BIDS

EEG2BIDS Wizard: a tool for converting raw EEG and iEEG data into the BIDS standard data structure, prepared for LORIS (Longitudinal Online Research and Imaging System).
MIT License
7 stars 10 forks source link

Adding support for .set and .mff files #83

Closed jesscall closed 2 years ago

jesscall commented 2 years ago

Brief summary of changes

This PR adds support to the Wizard for handling .set and .mff files.

Workflow for HBCD purposes:

  1. A user must select the file format they wish to run through the Wizard.
  2. For MFF, the user can upload multiple MFF directories. These should all come from the same task, just different runs.
  3. The MFF files are converted into SET by launching a MATLAB executable (./tools/mff_to_set.exe).
  4. The generated SET files will be used by the Wizard to begin BIDS-ification.

Capture d’écran (5)

Caveats + Dependencies:

Testing instructions (if applicable)

  1. For Windows: Contact @jesscall to get a hold of a testing packet.
  2. For other platforms, test functionality of EDF and SET file BIDS-ification without MFF conversion (not supported)

Link(s) to related issue(s)

jesscall commented 2 years ago

Notes on debugging the build for Windows:

Need to downgrade the following packages to specified versions:

Jinja2==2.11.3
MarkupSafe==2.0.1

I'm having issues with the MFF_to_SET.exe to launch in the Windows build. This is odd because I can get it to work if I end the app's python service and manually restart it.

Seems to be related to pyinstaller & subprocess - I tried some solutions outlined in this post, but it seems to break functionality of the mff conversion on the dev build.

Will continue to debug, but I think I might have to find a different implementation for launching the exe.

jesscall commented 2 years ago

Re-issued to new HBCD Wizard repo - https://github.com/aces/hbcd-eeg2bids/pull/1

Closing this PR