hipercog / ctap

Computational Testing for Automated Preprocessing - a Matlab toolbox extending EEGLAB functionality for batch processing of EEG
Other
22 stars 10 forks source link

Require 'event.type' field to be numeric? #25

Open zenBen opened 6 years ago

zenBen commented 6 years ago

While analysing EEG for a project that aims to use ERPLab, am discovering that it seems common to require event.type to be numeric: ERPLab requires it, and the trigger codes created directly by amplifiers are numeric, so data converted to EEGLAB format often has numeric event.type field by default.

This seems unnecessarily restrictive to me, but in order to be compatible, it might be an option for CTAP to enforce a numeric .type field, and use .label field for descriptive text, such as 'ctapeek'? The issue then becomes finding all the places in CTAP where some code adds or checks events with a char .type field, and refactoring.

See also Issues #24 and #13