change units for ScreenSize and EyeTrackerDistance to meter

bids-standard · Oct 25, 2023 · 3e510e1 · 3e510e1
1 parent 1736593
commit 3e510e1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/eye2bids/config/metadata.yml b/eye2bids/config/metadata.yml
@@ -2,14 +2,15 @@
 # manual_metadata
 # Please find the BIDS specification here: https://bids-specification--1128.org.readthedocs.build/en/1128/modality-specific-files/eye-tracking.html
 # Please go through the specification and fille the fields below according to the data types given in the specification.
+# !Note that ScreenSize and EyeTrackerDistance units are in meter, not centimeter!
 
 # REQUIRED, fill according to BIDS specification (!the converter will not run successfully if you leave this empty!):
 SampleCoordinateUnits: !!str
 SampleCoordinateSystem: !!str
 EnvironmentCoordinates: !!str
 ScreenDistance: !!int
 ScreenRefreshRate: !!int
-ScreenSize: [!!float]
+ScreenSize: [!!float] 
 
 # Recommended (leave empty if not available but put information you want to share with your dataset here!):
 SoftwareVersion: !!str

diff --git a/tests/data/metadata.yml b/tests/data/metadata.yml
@@ -9,7 +9,7 @@ SampleCoordinateSystem: gaze-on-screen
 EnvironmentCoordinates: top-left
 ScreenDistance: 60
 ScreenRefreshRate: 60
-ScreenSize: [38.6, 29]
+ScreenSize: [0.386, 0.29]
 
 # Recommended (leave empty if not available but put information you want to share with your dataset here!):
 SoftwareVersion: SREB1.10.1630 WIN32 LID:F2AE011 Mod:2017.04.21 15:19 CEST