-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
One modality to rule them all? #1324
Comments
Dear Kay, You are absolutely right, it would be great to clarify this in more detail across multiple modalities. For MOTION-BIDS we currently plan to advise using the scans.tsv to synchronize between multiple recordings: Best, |
This is a start, but not really the solution. As a side note.... I would strongly lobby to make the |
I hesitate to open this can of worms, but as more auxiliary modalities (e.g., motion, eye-tracking) are incorporated into BIDS, the use of multiple modalities in conjunction for analysis becomes problematic for BIDS, and this is of concern for downstream users of the data.
As far as I can tell, each modality is treated completely independently of each other with no standardized information about how they sync up. I think the motion BEP included time-stamps at one point (I don't know if it was in the final version being incorporated into BIDS), but this is insufficient for analysis in conjunction with modalities such as EEG since the time clocks don't run evenly and these recordings can run for a reasonable length of time.
I would like to raise this issue for consideration.... Does anyone have any thoughts on how this might be addressed in BIDS?
Is anyone interested on working on this?
The only idea that I have thought of so far is to have a "Sync" modality whose data consists of the sync markers for the individual data streams that could be synchronized downstream for analysis (something like available from LSL).
This would preserve the current model of modality independence, and allow the actual imaging data to be as is, but allow tools to synchronize for downstream analysis.
The text was updated successfully, but these errors were encountered: