bids-standard / eye2bids

Convert eyetracking data to a BIDS compatible format (BEP20)
MIT License
5 stars 6 forks source link

ensure continuous recording #69

Open Remi-Gau opened 5 months ago

Remi-Gau commented 5 months ago

ensures that the file generated have continuous recording

julia-pfarr commented 5 months ago

What does that mean exactly? I'm sorry, I've always had troubles understanding this. Does that have to do with the timestamps? The samples do have continuous recording (=_recording-

Is that what you mean? (asking because I'm currently working on the code for the _physioevents.tsv.gz file)

Remi-Gau commented 5 months ago

Yes the idea would be to pad physio.tsv.gz with rows of n/a for missing time stamps.

julia-pfarr commented 3 months ago

Ok, yes, I discussed this in our last meeting with Oscar and Martin. So the data (=physio.tsv.gz) should have continuous recording and I still need to modify the edf2bids code that it checks for continuity and if not fill with n/a rows. The events file however (=physioevents.tsv.gz) should not be continuous.

julia-pfarr commented 3 months ago

todo: add function to check the samples for continuous recordings; if False fill missing rows with continuous timestamps and nans for coordinates/pupil

Remi-Gau commented 4 minutes ago

OK I need to make sure that we have a test that actually make sure we throw a warning or an error if the output file does not have a continuous timestamp