search

mne-tools / fiff-constants

Bookkeeping and documentation of FIFF file format constants
4 stars 12 forks source link

ENH: Add Elekta files #4

Closed larsoner closed 6 years ago

larsoner commented 6 years ago

@agramfort should we incorporate our existing MNE constants in these files? It seems weird now to have separate files for the Elekta and MNE ranges.

agramfort commented 6 years ago

I fear it would be slow to parse these files in Python in every mne-python import.

what do you think of a unit test that checks that these files are consistent with mne-python constants.py ?

larsoner commented 6 years ago

Sure. It should probably live in mne-python, though. That way it is done continuously over there.

larsoner commented 6 years ago

WDYT about moving the _MNE.txt data into the main files now?

larsoner commented 6 years ago

Talking to the MEGIN folks, the MNE files should stay separate. In it goes!