-
Notifications
You must be signed in to change notification settings - Fork 169
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
[ENH] BEP 020 Eye Tracking #1128
base: master
Are you sure you want to change the base?
Changes from 180 commits
a73a9ea
b43e979
ac14d18
3316ef5
033a948
971ac2a
6236f83
1d71719
131b75d
b7f4d5d
5aa62f7
686a5f3
4a46c9f
0388cc1
e4f72c9
19d4b6c
7b6721d
6a71758
7941bab
f1c255b
2ffe348
e7658c9
3d2bf96
0369d4e
53e80f1
9f87572
d19eaf8
d222325
7c3a071
53d7894
0cd1366
108c18c
df50fd8
c90de08
e01eed1
5cc22ba
647af5e
77bf482
141e480
8c7618f
d678149
01332e5
b89469e
8fc5f0f
6fce90b
0009d63
dc793a3
a5f7cfc
2f81069
63352df
564a206
d6e8aed
42f4527
7ca6a42
7d435de
ea1d18f
49ec9e9
7fb149c
7a55abd
e3f74f8
9df5c68
c45f480
edc7c6f
518d8f6
8549d0e
f194526
0e0f800
15f817e
171eff9
d0c919a
39deaae
b621542
c76af5b
70c5edb
9f223a1
a9bfaaf
4c8327b
c6a8354
81d960e
1796cec
457e339
a0410f8
e4c8ad8
f063523
720311c
7868f57
8269cd3
c4c342b
752e38d
893911c
d802ab2
af48333
37f7cd1
9ecb9ea
e286064
e513efa
a414937
6f535cb
3ac5ca5
a31fbdd
17f296d
691ec27
a4af030
c9787a8
a0acebe
f96f383
08250d3
95d6dbf
2169021
9314fc0
8e01771
1aeeff5
2b85072
754f1c0
205a35f
cda8874
8be9429
e6ef2d6
84b869a
0ca61b4
53fcd97
4c7a30f
6c2b2f5
79fd36b
edc81bc
fcaaeaa
0bb5e41
c2ca15b
632eca2
75c0baa
05c8227
cbd2da7
8f90b33
a1126da
e728546
87e93c1
ef49458
1d05d11
2920134
3636333
aeac149
9df5c4e
b7d19b2
8b2e7e5
6968915
b56d7e0
a618326
57b1737
12b1abc
c8d8fc3
01f820d
61a1f31
ea5cc44
1e188a0
366cbc9
3b71c1f
e209833
e9fcd85
6f0d8fb
d16fd64
5ef41f4
311b645
90b9fba
51d7ae9
239f93c
f34589c
3606735
4aed597
3bbe59c
3beafc6
8c95b8c
96d984e
f57f393
8b6d458
6aeb1c2
e92027c
26c463a
a6a1c9d
0ad7fb1
a5ea212
6e787b6
9e697d2
09b40d7
cce40a0
19dab86
e222088
edaeace
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -162,6 +162,34 @@ duration: | |
type: number | ||
unit: s | ||
minimum: 0 | ||
x_coordinate: | ||
name: x_coordinate | ||
display_name: Gaze x-coordinate | ||
description: | | ||
Gaze position x-coordinate of the recorded eye, in the coordinate | ||
units specified in the corresponding metadata sidecar. | ||
type: number | ||
y_coordinate: | ||
name: y_coordinate | ||
display_name: Gaze y-coordinate | ||
description: | | ||
Gaze position y-coordinate of the recorded eye, in the coordinate | ||
units specified in the corresponding metadata sidecar. | ||
type: number | ||
pupil_size: | ||
name: pupil_size | ||
display_name: Pupil size | ||
description: | | ||
Pupil size or area of the recorded eye, in the | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are we recording the particular type of pupil size that's being recorded here ('diameter' v/s 'area')? The data that comes out of Eyelink is always in 'arbitrary units' (Eyelink documentation: page 106, 4.4.3) and the actual data type is stored in the settings (Eyelink documentation: page 25 'Pupil Size Data'). There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This could be another settings, do you feel like adding another entry to the table called something like pupil_type ? |
||
units specified in the corresponding metadata sidecar. | ||
mszinte marked this conversation as resolved.
Show resolved
Hide resolved
|
||
type: number | ||
et_timestamp: | ||
name: timestamp | ||
display_name: Eye-tracking recording timestamp | ||
description: | | ||
Timestamp issued by the eye-tracker indexing the continuous recordings corresponding to | ||
the sampled eye. | ||
type: number | ||
filename: | ||
name: filename | ||
display_name: Filename | ||
|
@@ -273,6 +301,13 @@ mapping: | |
description: | | ||
Corresponding integer label in the standard BIDS label lookup. | ||
type: integer | ||
message: | ||
name: message | ||
display_name: Message | ||
description: | | ||
Brief free-text description of a message (for example in a log generated | ||
by a device), or other information of interest. | ||
type: string | ||
material: | ||
name: material | ||
display_name: Electrode material | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As mentioned yesterday in the meeting: only for on-screen eyetracking.
Makes the validation of those a bit more complicated (one extra If).
I wonder if we should not have a similar warning in the eyetracking section, or at least a link to the task page.