bids-examples icon indicating copy to clipboard operation
bids-examples copied to clipboard

Draft: Face13 updates

Open Andesha opened this issue 2 years ago • 5 comments

This PR is currently a draft, however, the Face13 dataset has been updated to include HED tags!

There will be more commits to updating events files but this is the start.

Other minor fixes include updates to the coordsystem files, etc

cc @christinerogers

Andesha avatar Jun 14 '23 14:06 Andesha

Marked as draft so it's obvious to reviewers. There's a "Ready for review" button you can press whenever you want to turn it green again.

effigies avatar Jun 14 '23 14:06 effigies

this is contributed for EEGNet discussions around HED tags, how derivatives are evolving, etc.

christinerogers avatar Jun 16 '23 22:06 christinerogers

The events.tsv files still have HED columns that duplicate what is in the JSON file at the top level. It should be one or the other.

Also, I thought this dataset might have SCORE annotations, but I didn't see any in the example. Could you point me in the right direction on this? The tags I saw were all from the standard library, in which case the HEDVersion should just be "8.1.0" or better "8.2.0" since that is the latest.

VisLab avatar Jun 23 '23 13:06 VisLab

The validation seems to be failing because of large files. You will want to cutoff the number of events in the events.tsv file. If you remove the HED columns in the events.tsv and just keep the top-level JSON sidecar, you may be able to squeak by with full files.

VisLab avatar Jun 23 '23 13:06 VisLab

@VisLab just a note: it appears as though the BIDS validator is struggling to retrieve partnered schemas...

image

Andesha avatar Sep 26 '23 15:09 Andesha