DNM: Remove 10-20 from landmark description #117
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR tries to clarify the terminology used in
landmarkLabels
. It is in follow up to an email between me and the BUNPC.First, the old text referred to
probe.landmark
, which does not exist. I assume this was meant to refer toprobe.landmarkPos2D
andprobe.landmarkPos3D
, so I addedprobe.landmarkPos
to refer to both 2 and 3D.Second, I make a change to the landmarkLabels text as I believe it conflated "landmarks" and "standardized placement systems".
Landmarks are positions on the skull that are used to guide sensor placement. E.g. Nasion, Inion, LPA, RPA
Standardized placement systems specify locations on the head to place sensors. E.g. 10-20
[1] R. Oostenveld and P. Praamstra. The five percent electrode system for high-resolution EEG and ERP measurements. Clin Neurophysiol, 112:713-719, 2001.
[2] Fieldtrip
[3] BIDS
[4] MNE