search

mne-tools / mne-nirs

Process Near-Infrared Spectroscopy Data in MNE
https://mne.tools/mne-nirs/
BSD 3-Clause "New" or "Revised" License
84 stars 35 forks source link

SNIRF writer should not require birthdate #348

Closed rob-luke closed 3 years ago

rob-luke commented 3 years ago

Describe the bug

The SNIRF writer is requiring a birthdate. But this is only a recommended field for SNIRF.

Steps to reproduce

Load a file without birthdate, then try to write it back to disk.

Expected results

It writes to disk

Actual results

It crashes and points to lines https://github.com/mne-tools/mne-nirs/blob/341a2f38a7bef1899f473db52eda4197627c9c81/mne_nirs/io/snirf/_snirf.py#L74-L80

rob-luke commented 3 years ago

Closed in #349