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

[EEG-BIDS] Add support for events.json file & HED Tags #768

Closed
wants to merge 1 commit into from

Conversation

jesscall
Copy link
Contributor

@jesscall jesscall commented Mar 8, 2022

Add support to bids_import for *events.json file & HED Tags

Scope

This PR refactors how the bids_import handles event files. It adds support for an event.json file in which HED tags can be stored. The PR edits the fetching and inserting of event files and adds an archiving feature similar to the annotation files.

Information

Please reference this documentation for more info on HED Tags in BIDS

Testing

  1. The SQL patches from [EEG] Database Architecture for HED Tags Loris#8036 should be sourced before attempting to test these changes.
  2. Place the sample *events.json file at the root of the RB BIDS directory.
  3. Attempt to test the changes in this PR
  4. Try putting the *events.json file in a subject directory (and renaming) and make sure the import still runs as expected.

@jesscall jesscall added A-BIDS Area: BIDS. Issues and pull requests related to BIDS and the BIDS import pipeline A-EEG Area: EEG. Issues and pull requests related to EEG / iEEG processing labels Mar 8, 2022
@jesscall
Copy link
Contributor Author

jesscall commented Mar 9, 2022

oops, pushed to wrong repo -

@jesscall jesscall closed this Mar 9, 2022
@jesscall jesscall deleted the 2022_03_07_hed_tag_support branch March 9, 2022 18:15
@cmadjar cmadjar added this to the N/A milestone Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-BIDS Area: BIDS. Issues and pull requests related to BIDS and the BIDS import pipeline A-EEG Area: EEG. Issues and pull requests related to EEG / iEEG processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants