-
Notifications
You must be signed in to change notification settings - Fork 18
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
Coordinate system #51
Comments
For now we circumvent it by putting "Other" as EEGCoordinateSystem and "EEGLAB" in EEGCoordinateSystemDescription. Once EEGLAB is accepted we'll change EEGCoordinateSystem to "EEGLAB". |
Before and after exporting my dataset into BIDS, in EEGLAB the electrodes were represented according to the ALS system but not according to the ARS system and the units were not given in mm since the spherical radius was 1. When I checked the sample data from EEGLAB 2020_0, the units seemed to be in mm as the spherical radius was about 88, but the electrodes were also represented according to the ALS. Thus, I have two questions:
|
To do:
|
Any news on this @arnodelorme @cll008? The web page really doesn't have to be big or shiny --> a couple of sentences describing the coordinate system will suffice! Perhaps in the meantime these lines: Can be changed so that it says: That'd be more helpful than just |
Thanks for following up @sappelhoff . I wrote a draft of a page but Arno and I are checking in with Scott on some things. |
@arnodelorme @cll008 have you seen my comment in bids-standard/bids-specification#701 (comment) ? We basically just need a documentation page for the |
Thanks for the follow up @sappelhoff. Arno just made the webpage live. The coordinate descriptions are correct but we are making some more edits to the rest of the text. |
That's looking good! I'll prepare two PRs (one to the BIDS spec, one to the BIDS validator) and will tag you in it so that you can review. |
We export in the ARS system, but BIDS only compatible with the RAS system now (see also this post below)
bids-standard/bids-specification#701
Once the issue is resolved:
The text was updated successfully, but these errors were encountered: