search

sccn / EEG-BIDS

21 stars 17 forks source link

bids_export.m fix for recording duration (line 867) #151

Closed drpedapati closed 3 months ago

drpedapati commented 1 year ago

Great code Arno- working on a front end wrapper for multisite trials.

On line 867 tInfo.RecordingDuration = EEG.pnts/EEG.srate;

which is identical to tInfo.EpochLength = EEG.pnts/EEG.srate;

I think it should be something like: tInfo.RecordingDuration = (EEG.pnts/EEG.srate)*EEG.trials;

Doesn't require a separate issue, but for taskname validation (short, no spaces, or specials) wrapping it in: matlab.lang.makeValidName() may help.

arnodelorme commented 1 year ago

Multiple trial is good but is there a definition somewhere?

Yes, that sounds right tInfo.RecordingDuration = (EEG.pnts/EEG.srate)*EEG.trials;