Skip to content
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

HED-SCORE example of EEG and iEEG data with annotations #467

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

dorahermes
Copy link
Member

This BIDS example has EEG and iEEG data with annotations from the HED-SCORE library schema. It includes a subject with EEG data where artifacts are annotated, a subject with EEG data where seizures are annotated and a subject with iEEG data during photic stimulation was done and this is annotated as a modulator. Examples are provided of artifacts or seizures only observed in certain channels using the channel column in the _events.tsv file.

This example uses the recently released HED-SCORE version 2.0.0 and replaces pull request #324 .

It passes the BIDS online validator, comment welcome!

@VisLab @christinerogers

@christinerogers
Copy link

Hi @dorahermes I'm noticing bids-validator errors in the checks. Is this something expected / unexpected / easily solvable?

@effigies
Copy link
Contributor

effigies commented Oct 3, 2024

@dorahermes
Copy link
Member Author

Thank you so much for checking! I am not sure I can reproduce the error, as I did not get any errors using the online validator, except for the fact that the .edf files are empty, which I believe is required for the BIDS examples.

image

@christinerogers
Copy link

Reporting back from the Maintainers meeting -- A fix is on the way from the maintainers group since this is impacting lots of datasets, but in the meantime per @effigies :
Since this is passing on a few branches includingmaster-deno and 'master' this should be good to merge.
I can approve on that basis @dorahermes

@effigies effigies merged commit bcd3cb9 into bids-standard:master Oct 3, 2024
7 of 10 checks passed
Copy link

@christinerogers christinerogers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approving

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants