-
Notifications
You must be signed in to change notification settings - Fork 163
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
electrodes.tsv for simultaneous MEG/EEG data #1550
Comments
The spec says:
It's not clear to me whether "EEG sensor coordinates" refers to the |
Hi, Thank you very much for the reply. To check if it is pure MEG or MEG/EEG data, the ChannelFile is sufficient. In the ChannelFile it is distinguished whether it is MEG, EEG, ECG, etc. channel. However, I have direct experience here only with the Neuromag MEG system and for other systems only second-hand information. I hope that could help. Greetings, |
cc @robertoostenveld I think you would be a definitive authority here. |
We have concurrent iEEG/EEG data that are similarly recorded in one ieeg data file. In the _channels.tsv we include EEG and SEEG electrodes. The _electrodes.tsv has the corresponding electrode positions for each ieeg electrode, but does not contain the EEG positions or specify the type again (for an example, see subject 2 in doi:10.18112/openneuro.ds004457.v1.0.2). |
In a MEG dataset there can be EEG channels (or EOG, or ECG, or iEEG) for which the electrodes are not specified, but in the presence of EEG channels I agree that For the validator I recommend making it conditional on the EEGChannelCount in the |
Okay. So for the legacy validator, let's simply make Should it be disallowed (or discouraged) if |
Interested parties, please review: |
Thanks for the effort. |
Okay, so if |
I have a questions about these two comments by @robertoostenveld and @MatthiasSure:
and
in both of these cases we are talking about iEEG data. Is it true that iEEG data may be collected simultaneously with MEG data? And is this the problem that you are currently dealing with @MatthiasSure? Because if that is the case, the PR by @effigies in #1555 needs to be generalized from what it currently addresses (only EEG) to also encompass iEEG. Furthermore, if we are fine with this solution (and it does look good to me so far), then perhaps we should add a section about
Finally, for the record, I want to provide my interpretation of this problem mentioned above:
I think the original intent here was that EEG (or iEEG, ...) positions were supposed to be specified in the MEG specific file formats that specify sensor locations. For more information, see: and in particular see the information here: https://bids-specification.readthedocs.io/en/latest/modality-specific-files/magnetoencephalography.html#head-shape-and-electrode-description-_headshapeextension quoting:
having that said, I think a potential duplication of that information in PS: this discussion also lead me to identify a potential MEG spec bug: |
yes exactly. In my case, these are electrodes for deep brain stimulation which are externalized and connected to the EEG amplifier of the Neuromag MEG and thus recorded simultaneously with the MEG signal. The signal is therefore stored in the fif file as an EEG signal but is technically an iEEG signal. In the further proceeding, I will then label the channels concerned as SEEG or iEEG. |
Okay, thanks @MatthiasSure -- so you technically have the electrode positions in your Is there a way to change the channel type within the I am asking these questions to see whether we have actually don't have a problem here, and whether the original intent (IMHO) of the spec suffices as a solution:
(see my comment above for crosslinks to context and more info) @robertoostenveld do you agree with my comment from above? |
It sounds like there is not a problem here. I believe #1555 and bids-standard/bids-validator#1738 can move ahead. Or are we coming around to it should always be OPTIONAL for MEG? |
I'll try to come back to this soon (around 1 week), please wait with going ahead, I want to give this a double check. |
Is this the current state? Where does it say that? I think
In the current state of the spec, I'm okay with allowing this from now on, ... although I still think that the original intent of the spec was that in an MEG sesion, all data recorded from electrodes via the same system as the MEG signal, should go into the MEG specific file formats like directly in the
IMHO it should always be optional for MEG, EEG, iEEG.
+1 to move ahead with the legacy validator PR. However, the schema PR (#1555) needs small adjustments and an addition to the spec text, I think (will comment there). |
Regarding
Please note that afaik the neuromag system is the only with a well-integrated electrode acquisition strategy (using the same software as for MEG acquisition) and thereby the only that stores electrode information in the fif file. The others record and store it with separate systems in separate files. My preference would therefore be for |
Hi,
Back to this question:
The position of subcortical electrodes as used for LFP recordings via DBS electrodes is set to 0. However, using methods for the reconstruction of the location of electrodes it is possible to add this information later. That is what I'm doing. |
Thanks! I think we're all on the same page now and we can proceed to finish up the PR in #1555 and eventually merge it + merge the accompanying legacy bids-validator PR right now: https://github.com/bids-standard/bids-validator/pull/1738 |
I think that the _electrodes.tsv file should be RECOMMENDED (or REQUIRED) for iEEG data. Sharing iEEG data without electrode positions is like sharing voxel timeseries that are randomly shuffled without labeling, where you may be able to use a drawing or photograph with half legible labels to understand where 256 dots were positioned. Making _electrodes.tsv fully OPTIONAL for iEEG data will result in datasets that are being shared that are not interpretable or reusable. I am noting that in the original iEEG-BIDS extension on google doc, the _electrodes.tsv file was listed as REQUIRED for iEEG data, which is what the iEEG community agreed on (many iEEG community members are clinicians and are not on github). |
Hello all,
I want to bring MEG and LFP data recorded over the same amplifier into BIDS format.
Since the data is in one
fif
file (from the Neuromag system), everything is stored under/meg
.However, with the
*electrodes.tsv
it comes to an error message:with the BIDS validator.
In the specification description, I could not recognize a section to the
*electrodes.tsv
with MEG.Is this, therefore, not provided by MEG, or is there a solution for this problem?
Many greetings and thanks,
Matthias
The text was updated successfully, but these errors were encountered: